Skip to content

Conversation

sergio-mena
Copy link
Contributor

Changes in preparation for the first release candidate (RC) of v0.38.0:


PR checklist

  • Tests written/updated
  • Changelog entry added in .changelog (we use unclog to manage our changelog)
  • Updated relevant documentation (docs/ or spec/) and code comments

@sergio-mena sergio-mena added the release Pull requests that facilitate cutting a release label Jun 5, 2023
@sergio-mena sergio-mena self-assigned this Jun 5, 2023
@sergio-mena sergio-mena requested a review from a team as a code owner June 5, 2023 17:43
Copy link
Member

@hvanz hvanz left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@thanethomson thanethomson left a comment

Choose a reason for hiding this comment

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

One request re: the golangci-lint changes, but otherwise LGTM, thanks @sergio-mena!

@@ -31,7 +31,7 @@ jobs:
go.sum
- uses: golangci/golangci-lint-action@v3
with:
version: latest
version: v1.52.0
Copy link
Contributor

Choose a reason for hiding this comment

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

Could we please avoid pinning golangci-lint to a specific version?

I've submitted #922 to fix the lints on the v0.38.x branch.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok, will revert my commit and wait for #922 to be merged

Copy link
Contributor

Choose a reason for hiding this comment

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

Done!

Makefile Outdated
@@ -231,7 +231,7 @@ format:

lint:
@echo "--> Running linter"
@go run github.com/golangci/golangci-lint/cmd/golangci-lint@latest run
@go run github.com/golangci/golangci-lint/cmd/golangci-lint@v1.52.0 run
Copy link
Contributor

Choose a reason for hiding this comment

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

Same as previous comment.

@sergio-mena sergio-mena mentioned this pull request Jun 5, 2023
3 tasks
@sergio-mena sergio-mena merged commit 1dbb8ca into v0.38.x Jun 5, 2023
@sergio-mena sergio-mena deleted the release/v0.38.0-rc1 branch June 5, 2023 20:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release Pull requests that facilitate cutting a release
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants