Skip to content

Remove unused hasInternalPrefix function causing lint failure#10774

Merged
pelikhan merged 2 commits intomainfrom
copilot/fix-lint-go-workflow-again
Jan 20, 2026
Merged

Remove unused hasInternalPrefix function causing lint failure#10774
pelikhan merged 2 commits intomainfrom
copilot/fix-lint-go-workflow-again

Conversation

Copy link
Contributor

Copilot AI commented Jan 20, 2026

The lint-go workflow was failing due to an unused helper function hasInternalPrefix in pkg/workflow/compiler_orchestrator_test.go. This function was added in commit cb57a14 during validation error refactoring but never called.

Changes:

  • Removed unused hasInternalPrefix function (5 lines) from test file
// Removed:
func hasInternalPrefix(key string) bool {
	return len(key) > 2 && key[0] == '_' && key[1] == '_'
}

The golangci-lint unused checker now passes without errors.

Original prompt

Fix the failing GitHub Actions workflow lint-go
Analyze the workflow logs, identify the root cause of the failure, and implement a fix.
Format, lint, recompile
Job ID: 60843569956
Job URL: https://github.com/githubnext/gh-aw/actions/runs/21156947459/job/60843569956

Custom agent used: ci-cleaner
Tidies up the repository CI state by formatting sources, running linters, fixing issues, running tests, and recompiling workflows


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix failing GitHub Actions workflow lint-go Remove unused hasInternalPrefix function causing lint failure Jan 20, 2026
Copilot AI requested a review from pelikhan January 20, 2026 02:18
@pelikhan pelikhan marked this pull request as ready for review January 20, 2026 02:21
@pelikhan pelikhan merged commit 6bd7eb0 into main Jan 20, 2026
@pelikhan pelikhan deleted the copilot/fix-lint-go-workflow-again branch January 20, 2026 02:21
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.

2 participants