Skip to content

Run CI on pull requests only, point Dependabot at src/, float first-party actions - #38

Merged
imanimanyara merged 1 commit into
mainfrom
ci/triggers-and-currency
Sep 29, 2026
Merged

imanimanyara merged 1 commit into
mainfrom
ci/triggers-and-currency

Conversation

@imanimanyara

Copy link
Copy Markdown
Member
  • CI and CodeQL no longer trigger on push to main. That run happens after the change has landed and doubles every merge. pull_request, workflow_dispatch and CodeQL's weekly schedule remain.
  • Dependabot's gomod entry pointed at /, but the module lives in src/, so every Go update job failed.
  • First-party actions (actions/*, github/*) float on their major tag. Third-party actions stay SHA-pinned, per the org convention.
  • golangci-lint moves from v2.12.2 to v2.14.0, its latest release.

…arty actions

CI and CodeQL also ran on push to main, which runs after a change has landed and doubles every merge; pull_request, workflow_dispatch and CodeQL's weekly schedule remain. Dependabot's gomod entry looked for go.mod at the root while the module lives in src/, so every Go update job failed. First-party actions (actions/*, github/*) now float on their major tag and third-party ones stay SHA-pinned, per the org convention. golangci-lint moves to its latest release, v2.14.0.
Copilot AI balanced review requested due to automatic review settings September 29, 2026 08:44

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@imanimanyara
imanimanyara merged commit ee0cb0b into main Sep 29, 2026
9 checks passed
@imanimanyara
imanimanyara deleted the ci/triggers-and-currency branch September 29, 2026 09:41
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