Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Optimize GitHub action workflow scope #84

Closed
svengreb opened this issue Nov 18, 2021 · 0 comments · Fixed by #85
Closed

Optimize GitHub action workflow scope #84

svengreb opened this issue Nov 18, 2021 · 0 comments · Fixed by #85

Comments

@svengreb
Copy link
Owner

Currently all jobs are summarized in the ci workflow but not separated by their scope, e.g. only Node specific tasks. The workflow is also not optimized to only run when specific files have been changed which results in false-positive executions and wastes limited free tier and developer time.
Therefore the ci workflow will be optimized.

CI Node

A new ci-node workflow will…

@svengreb svengreb added this to the Next milestone Nov 18, 2021
@svengreb svengreb self-assigned this Nov 18, 2021
svengreb added a commit that referenced this issue Nov 18, 2021
Before all jobs were summarized in the `ci` workflow [1] but not
separated by their scope, e.g. only Node specific tasks. The workflow
was also not optimized to only run when specific files have been changed
which results in false-positive executions and wastes limited free tier
and developer time.
Therefore the `ci` workflow has been optimized.

>> CI Node

The new `ci-node` workflow…

- only runs when any `*.js`, `*.json`, `*.md`, `*.yaml` and `*.yml` file
  has been modified. This matches the lint-staged [2], Prettier and
  remark configurations. See the extensive GitHub action documentations
  about `on.<push|pull_request>.paths` [4] and the filter pattern cheat
  sheet [5] for more details.
- only runs for `ubuntu-latest` instead of a matrix with `macos-latest`
  and `windows-latest` since there is no platform specific code yet.
- use cache `npm` dependencies which is possible as of
  `actions/setup-node@v2.2.0` [3].

[1]: https://github.com/svengreb/tmpl/blob/0bb40e35/.github/workflows/ci.yml
[2]: https://github.com/svengreb/tmpl/blob/0bb40e35/lint-staged.config.js#L12-L13
[3]: https://github.com/actions/setup-node/releases/tag/v2.2.0
[4]: https://docs.github.com/en/actions/reference/workflow-syntax-for-github-actions#onpushpull_requestpaths
[5]: https://docs.github.com/en/actions/reference/workflow-syntax-for-github-actions#filter-pattern-cheat-sheet

GH-84
svengreb added a commit that referenced this issue Nov 18, 2021
Before all jobs were summarized in the `ci` workflow [1] but not
separated by their scope, e.g. only Node specific tasks. The workflow
was also not optimized to only run when specific files have been changed
which results in false-positive executions and wastes limited free tier
and developer time.
Therefore the `ci` workflow has been optimized.

>> CI Node

The new `ci-node` workflow…

- only runs when any `*.js`, `*.json`, `*.md`, `*.yaml` and `*.yml` file
  has been modified. This matches the lint-staged [2], Prettier and
  remark configurations. See the extensive GitHub action documentations
  about `on.<push|pull_request>.paths` [4] and the filter pattern cheat
  sheet [5] for more details.
- only runs for `ubuntu-latest` instead of a matrix with `macos-latest`
  and `windows-latest` since there is no platform specific code yet.
- use cache `npm` dependencies which is possible as of
  `actions/setup-node@v2.2.0` [3].

[1]: https://github.com/svengreb/tmpl/blob/0bb40e35/.github/workflows/ci.yml
[2]: https://github.com/svengreb/tmpl/blob/0bb40e35/lint-staged.config.js#L12-L13
[3]: https://github.com/actions/setup-node/releases/tag/v2.2.0
[4]: https://docs.github.com/en/actions/reference/workflow-syntax-for-github-actions#onpushpull_requestpaths
[5]: https://docs.github.com/en/actions/reference/workflow-syntax-for-github-actions#filter-pattern-cheat-sheet

GH-84
svengreb added a commit that referenced this issue Nov 18, 2021
Before all jobs were summarized in the `ci` workflow [1] but not
separated by their scope, e.g. only Node specific tasks. The workflow
was also not optimized to only run when specific files have been changed
which results in false-positive executions and wastes limited free tier
and developer time.
Therefore the `ci` workflow has been optimized.

>> CI Node

The new `ci-node` workflow…

- only runs when any `*.js`, `*.json`, `*.md`, `*.yaml` and `*.yml` file
  has been modified. This matches the lint-staged [2], Prettier and
  remark configurations. See the extensive GitHub action documentations
  about `on.<push|pull_request>.paths` [4] and the filter pattern cheat
  sheet [5] for more details.
- only runs for `ubuntu-latest` instead of a matrix with `macos-latest`
  and `windows-latest` since there is no platform specific code yet.
- use cache `npm` dependencies which is possible as of
  `actions/setup-node@v2.2.0` [3].

[1]: https://github.com/svengreb/tmpl/blob/0bb40e35/.github/workflows/ci.yml
[2]: https://github.com/svengreb/tmpl/blob/0bb40e35/lint-staged.config.js#L12-L13
[3]: https://github.com/actions/setup-node/releases/tag/v2.2.0
[4]: https://docs.github.com/en/actions/reference/workflow-syntax-for-github-actions#onpushpull_requestpaths
[5]: https://docs.github.com/en/actions/reference/workflow-syntax-for-github-actions#filter-pattern-cheat-sheet

GH-84
svengreb added a commit that referenced this issue Nov 18, 2021
Before all jobs were summarized in the `ci` workflow [1] but not
separated by their scope, e.g. only Node specific tasks. The workflow
was also not optimized to only run when specific files have been changed
which results in false-positive executions and wastes limited free tier
and developer time.
Therefore the `ci` workflow has been optimized.

>> CI Node

The new `ci-node` workflow…

- only runs when any `*.js`, `*.json`, `*.md`, `*.yaml` and `*.yml` file
  has been modified. This matches the lint-staged [2], Prettier and
  remark configurations. See the extensive GitHub action documentations
  about `on.<push|pull_request>.paths` [4] and the filter pattern cheat
  sheet [5] for more details.
- only runs for `ubuntu-latest` instead of a matrix with `macos-latest`
  and `windows-latest` since there is no platform specific code yet.
- use cache `npm` dependencies which is possible as of
  `actions/setup-node@v2.2.0` [3].

[1]: https://github.com/svengreb/tmpl/blob/0bb40e35/.github/workflows/ci.yml
[2]: https://github.com/svengreb/tmpl/blob/0bb40e35/lint-staged.config.js#L12-L13
[3]: https://github.com/actions/setup-node/releases/tag/v2.2.0
[4]: https://docs.github.com/en/actions/reference/workflow-syntax-for-github-actions#onpushpull_requestpaths
[5]: https://docs.github.com/en/actions/reference/workflow-syntax-for-github-actions#filter-pattern-cheat-sheet

Closes GH-84
@svengreb svengreb removed their assignment Nov 18, 2021
svengreb added a commit to svengreb/tmpl-go that referenced this issue Nov 20, 2021
Updated to `tmpl` version `0.10.0` [1] which comes with...

1. optimized GitHub action workflows for Go and Node [2] - this change
   has also already been resolved in GH-68 [5
   the `ci-go` GitHub Action workflow.
2. the regular Node package dependency & GitHub action version
   updates [3]
3. the migration to the Markdown style guide version `0.4.0` [4]

[1]: https://github.com/svengreb/tmpl/releases/tag/v0.10.0
[2]: svengreb/tmpl#84
[3]: svengreb/tmpl#86
[4]: svengreb/tmpl#76
[5]: #68

GH-70
svengreb added a commit to svengreb/tmpl-go that referenced this issue Nov 20, 2021
Updated to `tmpl` version `0.10.0` [1] which comes with...

1. optimized GitHub action workflows for Go and Node [2] - this change
   has also already been resolved in GH-68 [5
   the `ci-go` GitHub Action workflow.
2. the regular Node package dependency & GitHub action version
   updates [3]
3. the migration to the Markdown style guide version `0.4.0` [4]

[1]: https://github.com/svengreb/tmpl/releases/tag/v0.10.0
[2]: svengreb/tmpl#84
[3]: svengreb/tmpl#86
[4]: svengreb/tmpl#76
[5]: #68

Closes GH-70
arcticicestudio pushed a commit to svengreb/styleguide-markdown that referenced this issue May 10, 2022
Updated to `tmpl` version `0.11.0` [1], including the versions in
between starting from 0.10.0 [2]:

1. Optimized GitHub action workflow scope [3].
2. Updated Node.js packages & GitHub actions [4] [^1] [^2].
3. Opts-in the Dependabot version update configuration [5].

This also includes changes required for any linter matches.

[1]: https://github.com/svengreb/tmpl/releases/tag/v0.11.0
[2]: https://github.com/svengreb/tmpl/releases/tag/v0.10.0
[3]: svengreb/tmpl#84
[4]: svengreb/tmpl#86
[5]: svengreb/tmpl#94

[^1]: svengreb/tmpl#78
[^2]: svengreb/tmpl#83

GH-61
svengreb added a commit to svengreb/styleguide-markdown that referenced this issue May 10, 2022
Updated to `tmpl` version `0.11.0` [1], including the versions in
between starting from 0.10.0 [2]:

1. Optimized GitHub action workflow scope [3].
2. Updated Node.js packages & GitHub actions [4] [^1] [^2].
3. Opts-in the Dependabot version update configuration [5].

This also includes changes required for any linter matches.

[1]: https://github.com/svengreb/tmpl/releases/tag/v0.11.0
[2]: https://github.com/svengreb/tmpl/releases/tag/v0.10.0
[3]: svengreb/tmpl#84
[4]: svengreb/tmpl#86
[5]: svengreb/tmpl#94

[^1]: svengreb/tmpl#78
[^2]: svengreb/tmpl#83

GH-61
arcticicestudio pushed a commit to svengreb/styleguide-markdown that referenced this issue May 10, 2022
Updated to `tmpl` version `0.11.0` [1], including the versions in
between starting from 0.10.0 [2]:

1. Optimized GitHub action workflow scope [3].
2. Updated Node.js packages & GitHub actions [4] [^1] [^2].
3. Opts-in the Dependabot version update configuration [5].

This also includes changes required for any linter matches.

[1]: https://github.com/svengreb/tmpl/releases/tag/v0.11.0
[2]: https://github.com/svengreb/tmpl/releases/tag/v0.10.0
[3]: svengreb/tmpl#84
[4]: svengreb/tmpl#86
[5]: svengreb/tmpl#94

[^1]: svengreb/tmpl#78
[^2]: svengreb/tmpl#83

GH-61
arcticicestudio pushed a commit to svengreb/styleguide-markdown that referenced this issue May 12, 2022
Updated to `tmpl` version `0.11.0` [1], including the versions in
between starting from 0.10.0 [2]:

1. Optimized GitHub action workflow scope [3].
2. Updated Node.js packages & GitHub actions [4] [^1] [^2].
3. Opts-in the Dependabot version update configuration [5].

This also includes changes required for any linter matches.

[1]: https://github.com/svengreb/tmpl/releases/tag/v0.11.0
[2]: https://github.com/svengreb/tmpl/releases/tag/v0.10.0
[3]: svengreb/tmpl#84
[4]: svengreb/tmpl#86
[5]: svengreb/tmpl#94

[^1]: svengreb/tmpl#78
[^2]: svengreb/tmpl#83

GH-61
arcticicestudio pushed a commit to svengreb/styleguide-markdown that referenced this issue May 12, 2022
Updated to `tmpl` version `0.11.0` [1], including the versions in
between starting from 0.10.0 [2]:

1. Optimized GitHub action workflow scope [3].
2. Updated Node.js packages & GitHub actions [4] [^1] [^2].
3. Opts-in the Dependabot version update configuration [5].

This also includes changes required for any linter matches.

[1]: https://github.com/svengreb/tmpl/releases/tag/v0.11.0
[2]: https://github.com/svengreb/tmpl/releases/tag/v0.10.0
[3]: svengreb/tmpl#84
[4]: svengreb/tmpl#86
[5]: svengreb/tmpl#94

[^1]: svengreb/tmpl#78
[^2]: svengreb/tmpl#83

GH-61
arcticicestudio pushed a commit to svengreb/styleguide-markdown that referenced this issue May 12, 2022
Updated to `tmpl` version `0.11.0` [1], including the versions in
between starting from 0.10.0 [2]:

1. Optimized GitHub action workflow scope [3].
2. Updated Node.js packages & GitHub actions [4] [^1] [^2].
3. Opts-in the Dependabot version update configuration [5].

This also includes changes required for any linter matches.

[1]: https://github.com/svengreb/tmpl/releases/tag/v0.11.0
[2]: https://github.com/svengreb/tmpl/releases/tag/v0.10.0
[3]: svengreb/tmpl#84
[4]: svengreb/tmpl#86
[5]: svengreb/tmpl#94

[^1]: svengreb/tmpl#78
[^2]: svengreb/tmpl#83

GH-61
arcticicestudio pushed a commit to svengreb/styleguide-javascript that referenced this issue May 12, 2022
Updated to `tmpl` version `0.11.0` [1], including the versions in
between starting from 0.10.0 [2]:

1. Optimized GitHub action workflow scope [3].
2. Updated Node.js packages & GitHub actions [4] [^1] [^2].
3. Opts-in the Dependabot version update configuration [5].
4. Migrated to Markdown style guide version 0.4.0 [6].

This also includes changes required for any linter matches.

[1]: https://github.com/svengreb/tmpl/releases/tag/v0.11.0
[2]: https://github.com/svengreb/tmpl/releases/tag/v0.10.0
[3]: svengreb/tmpl#84
[4]: svengreb/tmpl#86
[5]: svengreb/tmpl#94
[6]: svengreb/tmpl#76

[^1]: svengreb/tmpl#78
[^2]: svengreb/tmpl#83

GH-86
arcticicestudio pushed a commit to svengreb/styleguide-javascript that referenced this issue May 12, 2022
Updated to `tmpl` version `0.11.0` [1], including the versions in
between starting from 0.10.0 [2]:

1. Optimized GitHub action workflow scope [3].
2. Updated Node.js packages & GitHub actions [4] [^1] [^2].
3. Opts-in the Dependabot version update configuration [5].
4. Migrated to Markdown style guide version 0.4.0 [6].

This also includes changes required for any linter matches.

[1]: https://github.com/svengreb/tmpl/releases/tag/v0.11.0
[2]: https://github.com/svengreb/tmpl/releases/tag/v0.10.0
[3]: svengreb/tmpl#84
[4]: svengreb/tmpl#86
[5]: svengreb/tmpl#94
[6]: svengreb/tmpl#76

[^1]: svengreb/tmpl#78
[^2]: svengreb/tmpl#83


Co-authored-by: Sven Greb <development@svengreb.de>

GH-86
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant