-
Notifications
You must be signed in to change notification settings - Fork 427
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
fix: golangci.yml to keep quality of codes #1296
Conversation
@sfc-gh-swinkler Could you please review this change? |
_, err = snowflake.ListTagAssociations(builder, db) | ||
if err != nil { | ||
return errors.Wrap(err, "error validating tag association") | ||
skipValidate := d.Get("skip_validation").(bool) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
im wondering where this came from. could you please rebase to main?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure! I think this came from other PR and I failed to operate merging other one. I'll check this carefully then fix around this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sfc-gh-swinkler Again, thank you for your review. I checked changes around these lines and found just my deleting blank two lines. Does this problem come from somewhere else...?
could you please rebase to main? theres some changes to the tag_association resource i am thinking probably came from an old PR |
@sfc-gh-swinkler Could you please rerun workflow? |
@mnagaa thank you for your contribution! |
Summary
golangci lint does not work because of including unnecessary indent in the
.golangci.yml
.Fixing the broken
.golangci.yml
and invalid expressions in the existing go files are done in this PR.Notice that linters commented out in the file yaml file need fix many files, but they should work in the future.