Skip to content

Conversation

@sudo-suhas
Copy link
Contributor

@sudo-suhas sudo-suhas commented Nov 7, 2022

  • Bump Go version {1.17 => 1.18}.
    • Ensure to fetch the latest version in 'build' and 'release'
      workflows to pull in the most up-to-date release so that security
      fixes can be included.
    • Bump the golangci-lint version {v1.42.1 => v1.50}. This is required
      to be able to lint source code with Go 1.18. Fails with 'undeclared
      name' errors otherwise.
  • Do not run 'go mod tidy' as this can modify the mod file. Changes to
    dependencies should be explicitly committed where required.
  • Modify release workflow as per
    https://goreleaser.com/ci/actions/#workflow.
  • Bump action versions:
    • actions/setup-go@{v2 => v3}. Enable the 'cache' option for caching
      dependency files and build outputs.
    • actions/checkout@{v2 => v3}.
    • golangci/golangci-lint-action@{v2 => v3}. Caching is enabled by
      default. Bump golangci-lint version {v1.42.1 => v1.50}, needed for
      Go 1.18.
  • Fix linting issues due to deprecation of ioutil package. Migrate to io
    and os packages instead.
  • Consistent formatting:
    • Checkout code first in all jobs across workflows with the name
      'Checkout code'.
    • Setup Go after 'Checkout code' with the name 'Set up Go'.

@sudo-suhas sudo-suhas force-pushed the ci-go-cache branch 3 times, most recently from 630b78b to da13fb0 Compare November 7, 2022 12:53
- Bump Go version {1.17 => 1.18}.
  - Ensure to fetch the latest version in 'build' and 'release'
    workflows to pull in the most up-to-date release so that security
    fixes can be included.
  - Bump the golangci-lint version {v1.42.1 => v1.50}. This is required
    to be able to lint source code with Go 1.18. Fails with 'undeclared
    name' errors otherwise.
- Do not run 'go mod tidy' as this can modify the mod file. Changes to
  dependencies should be explicitly committed where required.
- Modify release workflow as per
  https://goreleaser.com/ci/actions/#workflow.
- Bump action versions:
  - actions/setup-go@{v2 => v3}. Enable the 'cache' option for caching
    dependency files and build outputs.
  - actions/checkout@{v2 => v3}.
  - golangci/golangci-lint-action@{v2 => v3}. Caching is enabled by
    default. Bump golangci-lint version {v1.42.1 => v1.50}, needed for
    Go 1.18.
- Fix linting issues due to deprecation of ioutil package. Migrate to io
  and os packages instead.
- Consistent formatting:
  - Checkout code first in all jobs across workflows with the name
    'Checkout code'.
  - Setup Go after 'Checkout code' with the name 'Set up Go'.
@sudo-suhas
Copy link
Contributor Author

Before

image

(sample CI run from #432)

After

image

Copy link
Contributor

@StewartJingga StewartJingga left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice @sudo-suhas 🔥

@sudo-suhas sudo-suhas merged commit 6495614 into main Nov 7, 2022
@sudo-suhas sudo-suhas deleted the ci-go-cache branch November 7, 2022 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants