Skip to content

ci(gh-actions): fix gci import prefix; drop redundant tidy check#48

Merged
hyp3rd merged 1 commit intomainfrom
feat/secrets
Jan 12, 2026
Merged

ci(gh-actions): fix gci import prefix; drop redundant tidy check#48
hyp3rd merged 1 commit intomainfrom
feat/secrets

Conversation

@hyp3rd
Copy link
Owner

@hyp3rd hyp3rd commented Jan 12, 2026

  • go.yml: set default GCI_PREFIX to github.com/hyp3rd/sectools so gci groups local imports under the correct module.
  • lint.yml: remove the "Tidy check" step (go mod tidy + diff); this check already runs in the Go build workflow to avoid duplication and churn.
  • docs: remove template "Quick Start" section in README that referenced the starter repo and no longer applies.

- go.yml: set default GCI_PREFIX to github.com/hyp3rd/sectools so gci groups local imports under the correct module.
- lint.yml: remove the "Tidy check" step (go mod tidy + diff); this check already runs in the Go build workflow to avoid duplication and churn.
- docs: remove template "Quick Start" section in README that referenced the starter repo and no longer applies.
Copilot AI review requested due to automatic review settings January 12, 2026 15:20
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request updates CI configuration to reflect the correct repository context and remove redundant checks. The changes ensure the GCI import prefix matches the actual module path (github.com/hyp3rd/sectools) instead of the starter template path, eliminate duplicate tidy checks between workflows, and remove obsolete documentation that referenced the starter repository.

Changes:

  • Updated GCI_PREFIX default values in both GitHub Actions workflows from github.com/hyp3rd/starter to github.com/hyp3rd/sectools
  • Removed duplicate "Tidy check" step from lint.yml (already runs in go.yml)
  • Removed outdated Quick Start documentation section that referenced the starter repository

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
.github/workflows/lint.yml Updated GCI_PREFIX default and removed duplicate tidy check
.github/workflows/go.yml Updated GCI_PREFIX default to match actual module path
README.md Removed Quick Start section that was specific to the starter template

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@hyp3rd hyp3rd merged commit b28f899 into main Jan 12, 2026
21 checks passed
@hyp3rd hyp3rd deleted the feat/secrets branch January 19, 2026 15:53
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.

1 participant