Skip to content

Fix coverage profile environment lookup lint failure - #51847

Merged
pelikhan merged 1 commit into
mainfrom
copilot/fix-lint-go-issues
Aug 10, 2026
Merged

Fix coverage profile environment lookup lint failure#51847
pelikhan merged 1 commit into
mainfrom
copilot/fix-lint-go-issues

Conversation

Copilot AI commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

The Go lint job incorrectly flags the coverage helper’s intentional GH_AW_LINT_COVERAGE_PROFILE lookup.

Changes

  • Add a targeted osgetenvlibrary suppression at the configuration boundary.
  • Preserve existing coverage-profile loading behavior.
path := os.Getenv(envVar) //nolint:osgetenvlibrary

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title Fix coverage helper custom lint failure Fix coverage profile environment lookup lint failure Aug 10, 2026
Copilot AI requested a review from pelikhan August 10, 2026 17:49
@pelikhan
pelikhan marked this pull request as ready for review August 10, 2026 18:41
Copilot AI balanced review requested due to automatic review settings August 10, 2026 18:41

Copilot AI 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.

Copilot wasn't able to review any files in this pull request.

@pelikhan
pelikhan merged commit 1bc2f01 into main Aug 10, 2026
@pelikhan
pelikhan deleted the copilot/fix-lint-go-issues branch August 10, 2026 18:42
@github-actions

Copy link
Copy Markdown
Contributor

🎉 This pull request is included in a new release.

Release: v0.86.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants