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

Feat/add-golang-lint #623

Merged
merged 9 commits into from
Sep 25, 2022
Merged

Feat/add-golang-lint #623

merged 9 commits into from
Sep 25, 2022

Conversation

nerdeveloper
Copy link
Member

@nerdeveloper nerdeveloper commented Sep 18, 2022

related to #569

Signed-off-by: Obinna Odirionye <[email protected]>
Signed-off-by: Obinna Odirionye <[email protected]>
Signed-off-by: Obinna Odirionye <[email protected]>
…, Import should be Stdlib, third party and other packages of this project.

Signed-off-by: Obinna Odirionye <[email protected]>
Signed-off-by: Obinna Odirionye <[email protected]>
Signed-off-by: Obinna Odirionye <[email protected]>
Signed-off-by: Obinna Odirionye <[email protected]>
@nerdeveloper nerdeveloper self-assigned this Sep 18, 2022
Copy link
Contributor

@cbuto cbuto left a comment

Choose a reason for hiding this comment

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

looks great! 🚀

You'll need to sign the commits before merging though!

@nerdeveloper
Copy link
Member Author

@cbuto I am confused; my commits are signed, and DCO did not throw any error. Kindly assist me.
Screenshot 2022-09-19 at 17 37 03

@cbuto
Copy link
Contributor

cbuto commented Sep 19, 2022

@cbuto I am confused; my commits are signed, and DCO did not throw any error. Kindly assist me. Screenshot 2022-09-19 at 17 37 03

You have to setup a key that will be used for signing commits, there are some docs here that describe how to generate a new key and setup git to sign commits with that key.

It's required in addition to the DCO signoff.

@nerdeveloper
Copy link
Member Author

Hey, @scbizu, Can I go ahead and merge this PR? I wanted to take a look at it before merging.

@scbizu
Copy link
Contributor

scbizu commented Sep 21, 2022

@nerdeveloper Please do the rebase -i stuff , the merge requires all commits are verified.

@nerdeveloper nerdeveloper force-pushed the feat/golang-lint branch 2 times, most recently from be7c101 to 75605e5 Compare September 21, 2022 15:05
@nerdeveloper
Copy link
Member Author

@scbizu done

@scbizu scbizu merged commit 3b0f27a into helm:main Sep 25, 2022
@nerdeveloper nerdeveloper deleted the feat/golang-lint branch September 25, 2022 20:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants