Skip to content
This repository has been archived by the owner on Jul 5, 2024. It is now read-only.

Upgrade golint version to v1.45.2 #504

Merged
2 commits merged into from
Apr 5, 2022
Merged

Upgrade golint version to v1.45.2 #504

2 commits merged into from
Apr 5, 2022

Conversation

ghost
Copy link

@ghost ghost commented Apr 5, 2022

No description provided.

Signed-off-by: Mustafa Kara <mkara@pinchflat.dev>
@ghost ghost added the 2: Dev Review Requires review by a core committer label Apr 5, 2022
Signed-off-by: Mustafa Kara <mkara@pinchflat.dev>
@ghost ghost assigned spirosoik, isacikgoz, phoinixgrr and ghost and unassigned spirosoik, isacikgoz and phoinixgrr Apr 5, 2022
@ghost ghost requested review from phoinixgrr, spirosoik and isacikgoz April 5, 2022 14:33
Copy link
Member

@isacikgoz isacikgoz left a comment

Choose a reason for hiding this comment

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

Nice one!

Copy link
Contributor

@phoinixgrr phoinixgrr left a comment

Choose a reason for hiding this comment

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

🎖️ Great work!

@pfltdv Could you provide some context on 62ef3a8 ?

@ghost
Copy link
Author

ghost commented Apr 5, 2022

🎖️ Great work!

@pfltdv Could you provide some context on 62ef3a8 ?

@phoinixgrr, that line is not compatible with lintin rules. Somehow it was skipped on master.

Rule:

  error-strings: error strings should not be capitalized or end with punctuation or a newline (revive)

So removed dot at the end.

@ghost ghost added 4: Reviews Complete All reviewers have approved the pull request and removed 2: Dev Review Requires review by a core committer labels Apr 5, 2022
@ghost ghost merged commit 4d7dab5 into master Apr 5, 2022
@ghost ghost deleted the chore/fix-broken-golint-step branch April 5, 2022 15:06
@phoinixgrr
Copy link
Contributor

🎖️ Great work!
@pfltdv Could you provide some context on 62ef3a8 ?

@phoinixgrr, that line is not compatible with lintin rules. Somehow it was skipped on master.

Rule:

  error-strings: error strings should not be capitalized or end with punctuation or a newline (revive)

So removed dot at the end.

Awesome, Thanks for this @pfltdv 👍

This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
4: Reviews Complete All reviewers have approved the pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants