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 OS version matrix strategy for CI workflow #46

Closed
svengreb opened this issue Sep 25, 2020 · 0 comments · Fixed by #47
Closed

Optimize OS version matrix strategy for CI workflow #46

svengreb opened this issue Sep 25, 2020 · 0 comments · Fixed by #47

Comments

@svengreb
Copy link
Owner

The CI workflow currently uses a matrix strategy to run the lint-node job, but this should not be necessary for this repository. This can be improved to make the workflow run faster by avoiding unnecessary steps:

  • The lint-node job will be changed to only run on the currently latest stable Node version 14.x only on Linux because this repository is not focused on JavaScript but only runs Node based tools to lint other files within this repository.

These changes will also help to keep the required GitHub Action run minutes for the account of this repository as small as possible without wasting resources for unnecessary tasks.

@svengreb svengreb added this to the Next milestone Sep 25, 2020
@svengreb svengreb self-assigned this Sep 25, 2020
svengreb added a commit that referenced this issue Sep 25, 2020
Before the CI workflow used a matrix strategy to run the `lint-node`
job, but this was not necessary for this repository. It has been
improved to make the workflow run faster by avoiding unnecessary steps:

- The `lint-node` job has been changed to only run on the currently
  latest stable Node version `14.x` [1] only on Linux because this
  repository is not focused on JavaScript but only runs Node based tools
  to lint other files within this repository.

These changes also help to keep the required GitHub Action run minutes
for the account of this repository as small as possible without wasting
resources for unnecessary tasks.

[1]: https://github.com/nodejs/node/blob/master/doc/changelogs/CHANGELOG_V14.md#14.12.0

GH-46
@svengreb svengreb changed the title Optimize OS and Go version matrix strategy for CI workflow Optimize OS version matrix strategy for CI workflow Sep 25, 2020
svengreb added a commit that referenced this issue Sep 25, 2020
Before the CI workflow used a matrix strategy to run the `lint-node`
job, but this was not necessary for this repository. It has been
improved to make the workflow run faster by avoiding unnecessary steps:

- The `lint-node` job has been changed to only run on the currently
  latest stable Node version `14.x` [1] only on Linux because this
  repository is not focused on JavaScript but only runs Node based tools
  to lint other files within this repository.

These changes also help to keep the required GitHub Action run minutes
for the account of this repository as small as possible without wasting
resources for unnecessary tasks.

[1]: https://github.com/nodejs/node/blob/master/doc/changelogs/CHANGELOG_V14.md#14.12.0

Resolves GH-46
@svengreb svengreb removed their assignment Sep 25, 2020
svengreb added a commit to svengreb/tmpl-go that referenced this issue Sep 25, 2020
Adapted to "tmpl" version 0.4.0 [1] which includes a optimized OS
version matrix strategy for Node based tasks in the CI workflow [2] that
helps to keep the required GitHub Action run minutes for the account of
this repository as small as possible without wasting resources for
unnecessary tasks.

[1]: https://github.com/svengreb/tmpl/releases/tag/v0.4.0
[2]: svengreb/tmpl#46

GH-14
svengreb added a commit to svengreb/tmpl-go that referenced this issue Sep 25, 2020
Adapted to "tmpl" version 0.4.0 [1] which includes a optimized OS
version matrix strategy for Node based tasks in the CI workflow [2] that
helps to keep the required GitHub Action run minutes for the account of
this repository as small as possible without wasting resources for
unnecessary tasks.

[1]: https://github.com/svengreb/tmpl/releases/tag/v0.4.0
[2]: svengreb/tmpl#46

Resolves GH-14
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