Skip to content

Feat/add-golang-lint#623

Merged
scbizu merged 9 commits into
helm:mainfrom
nerdeveloper:feat/golang-lint
Sep 25, 2022
Merged

Feat/add-golang-lint#623
scbizu merged 9 commits into
helm:mainfrom
nerdeveloper:feat/golang-lint

Conversation

@nerdeveloper

@nerdeveloper nerdeveloper commented Sep 18, 2022

Copy link
Copy Markdown
Member

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

@cbuto cbuto left a comment

Copy link
Copy Markdown
Contributor

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
Copy Markdown
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

cbuto commented Sep 19, 2022

Copy link
Copy Markdown
Contributor

@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
Copy Markdown
Member Author

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

@scbizu

scbizu commented Sep 21, 2022

Copy link
Copy Markdown
Contributor

@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
Copy Markdown
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