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

Chore upgrade linter #751

Merged
merged 2 commits into from
Jun 12, 2024
Merged

Chore upgrade linter #751

merged 2 commits into from
Jun 12, 2024

Conversation

emcfarlane
Copy link
Contributor

Upgrades the linter version to the latest fixing small issues. For readability nolint global directives have been brought inline to avoid misuse. This fixes the issue brought up in #748. As a small cleanup the buf version has been brought to the latest.

Upgrades the linter version to the latest fixing small issues. For
readability nolint global directives have been brought inline to avoid
misuse. As a small cleanup the buf version has been bumped too.

Signed-off-by: Edward McFarlane <emcfarlane@buf.build>
@emcfarlane emcfarlane requested a review from jhump June 12, 2024 20:25
@emcfarlane emcfarlane marked this pull request as draft June 12, 2024 20:32
@emcfarlane emcfarlane force-pushed the ed/updateLint branch 3 times, most recently from 980263f to 9bb955e Compare June 12, 2024 20:59
Signed-off-by: Edward McFarlane <emcfarlane@buf.build>
@emcfarlane emcfarlane marked this pull request as ready for review June 12, 2024 21:21
@emcfarlane emcfarlane merged commit e87d966 into main Jun 12, 2024
12 checks passed
@emcfarlane emcfarlane deleted the ed/updateLint branch June 12, 2024 23:11
@jhump jhump added the ignore-for-release Exclude from release notes label Sep 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ignore-for-release Exclude from release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants