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

Fix Failing CI Tests #393

Merged
merged 5 commits into from
Mar 28, 2020
Merged

Fix Failing CI Tests #393

merged 5 commits into from
Mar 28, 2020

Conversation

jcudit
Copy link
Contributor

@jcudit jcudit commented Mar 25, 2020

This PR fixes minor errors that were causing linter failures during CI runs.

3 / 4 errors were removed but one remains that I suspect will go away after automation kicks in for the next release. See here for more details around the remaining failure.

cc https://github.com/terraform-providers/terraform-provider-github/pull/362 (fb14221)

`golangci-lint run` fais to execute because of missing vendored 
dependencies.
Jeremy Udit added 4 commits March 25, 2020 10:24
> SA5011: possible nil pointer dereference (staticcheck)
> SA1029: should not use built-in type string as key for value; define 
your own type to avoid collisions (staticcheck)
Refactor the error catching logic and be explicit about checking for a 
nil pointer.
@jcudit jcudit requested a review from paultyng March 25, 2020 15:37
@jcudit jcudit self-assigned this Mar 25, 2020
@jcudit jcudit added this to the v2.5.0 milestone Mar 27, 2020
Copy link

@mbfrahry mbfrahry left a comment

Choose a reason for hiding this comment

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

LGTM

@jcudit jcudit merged commit b45a9f8 into master Mar 28, 2020
@jcudit jcudit deleted the fix-linter-failures branch March 28, 2020 14:35
kfcampbell pushed a commit to kfcampbell/terraform-provider-github that referenced this pull request Jul 26, 2022
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.

2 participants