Skip to content

Fix dependabot build failures: migrate golangci-lint config to v2 - #35

Merged
geoffreywiseman merged 5 commits into
mainfrom
copilot/fix-dependabot-build-failures
Mar 7, 2026
Merged

geoffreywiseman merged 5 commits into
mainfrom
copilot/fix-dependabot-build-failures

Conversation

Copilot AI commented Mar 7, 2026 •

Copy link
Copy Markdown
Contributor

Dependabot bumped golangci-lint v1→v2, which removed the default: standard preset and broke the lint workflow. Also updates stale action versions (checkout@v3, setup-go@v3).

Linter config (golangci.yml)

  • Replaces default: standard + 35-item enable: list with default: all + 18-item disable: list
  • Each disabled linter is annotated with the reason; new linters are picked up automatically going forward

GitHub Actions workflows

  • actions/checkout@v3 → v4, actions/setup-go@v3 → v5

Code fixes (surfaced by broader default: all coverage)

  • interface{} → any
  • .Error() methods: fmt.Sprintf("x: %s", y) → "x: " + y
  • == 404 → == http.StatusNotFound
  • Package doc comments on client, format, main, mock
  • testRepoFullName constant extracted in client_test.go (was repeated 7×)

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/graphql
    • Triggering command: /usr/bin/gh gh pr view 35 --json number,title (http block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

This section details on the original issue you should resolve

<issue_title>Builds Failling</issue_title>
<issue_description>Recent builds have all failed -- I configured Dependabot and it looks like each PR created by Dependabot has failed checks, so I suspect there's something wrong with the build. Look into it and work on a solution.</issue_description>

Comments on the Issue (you are @copilot in this section)


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Copilot AI and others added 2 commits March 7, 2026 00:41
… to v2 format, update ci.yml action versions

Co-authored-by: geoffreywiseman <49935+geoffreywiseman@users.noreply.github.com>
…23 are EOL)

Co-authored-by: geoffreywiseman <49935+geoffreywiseman@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix failing builds caused by Dependabot updates Fix CI failures from Dependabot Go dependency bumps Mar 7, 2026
… v2 migration

Co-authored-by: geoffreywiseman <49935+geoffreywiseman@users.noreply.github.com>
Copilot AI changed the title Fix CI failures from Dependabot Go dependency bumps Fix Dependabot-triggered CI failures: upgrade Go toolchain and golangci-lint v1→v2 Mar 7, 2026
…x surfaced code issues

Co-authored-by: geoffreywiseman <49935+geoffreywiseman@users.noreply.github.com>
Copilot AI changed the title Fix Dependabot-triggered CI failures: upgrade Go toolchain and golangci-lint v1→v2 Migrate golangci-lint config to default: all with focused disable list Mar 7, 2026
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Mar 7, 2026
Copilot AI changed the title Migrate golangci-lint config to default: all with focused disable list Streamline golangci-lint config: default: all + focused disable list Mar 7, 2026
Copilot AI changed the title Streamline golangci-lint config: default: all + focused disable list Fix dependabot build failures: migrate golangci-lint config to v2 Mar 7, 2026
@geoffreywiseman
geoffreywiseman marked this pull request as ready for review March 7, 2026 01:15
@dosubot dosubot Bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Mar 7, 2026
@geoffreywiseman
geoffreywiseman merged commit 731a102 into main Mar 7, 2026
16 checks passed
@geoffreywiseman
geoffreywiseman deleted the copilot/fix-dependabot-build-failures branch March 7, 2026 01:16
@dosubot dosubot Bot added the bug Something isn't working label Mar 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Builds Failling

2 participants