Skip to content

fix(ci): skip commit-subject validation in the post-merge pipeline - #41

Merged
bougyman merged 1 commit into
mainfrom
issue-40
Aug 10, 2026
Merged

fix(ci): skip commit-subject validation in the post-merge pipeline#41
bougyman merged 1 commit into
mainfrom
issue-40

Conversation

@bougyman

Copy link
Copy Markdown
Member

Summary

Closes #40.

main.yaml's validate job calls ci.yaml as a reusable workflow, running both Test and Validate Commit Subjects post-merge. The commit-subject check only makes sense pre-merge (should this PR be mergeable) - by the time main.yaml runs, the commits are already merged, so re-checking their subjects is meaningless. Test still re-runs post-merge (verifies the actual merge result); commit-subject validation no longer does.

Added a skip_commit_validation workflow_call input on ci.yaml (default false, so its own pull_request/workflow_dispatch triggers are unaffected), and main.yaml's validate job now passes skip_commit_validation: true.

Test plan

  • Both edited workflow files validated with ruby -ryaml
  • Confirmed the if: inputs.skip_commit_validation != true expression safely defaults to running the job for every trigger type that doesn't explicitly pass the input (pull_request, workflow_dispatch) - inputs.skip_commit_validation evaluates to null there, and null != true is true

Validate Commit Subjects only makes sense pre-merge (should this PR be
mergeable) - by the time main.yaml's validate job runs, the commits are
already merged, so re-checking their subjects is meaningless. Test
still re-runs post-merge (verifies the actual merge result); commit-
subject validation no longer does, via a new skip_commit_validation
workflow_call input on ci.yaml.

Closes #40.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings August 10, 2026 00:17

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.

Pull request overview

Updates the post-merge workflow (main.yaml) to avoid re-running commit-subject validation after a PR is already merged, while keeping the reusable CI workflow (ci.yaml) behavior unchanged for its normal pre-merge triggers.

Changes:

  • Add a skip_commit_validation boolean input to the reusable CI workflow (ci.yaml) with a default of false.
  • Gate the “Validate Commit Subjects” job in ci.yaml behind if: inputs.skip_commit_validation != true.
  • Pass skip_commit_validation: true from main.yaml when invoking the reusable CI workflow post-merge.

Reviewed changes

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

File Description
.github/workflows/main.yaml Passes skip_commit_validation: true when calling the reusable CI workflow in the post-merge validations job.
.github/workflows/ci.yaml Introduces a workflow_call input and uses it to conditionally skip the commit-subject validation job.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@bougyman
bougyman merged commit 3d00a85 into main Aug 10, 2026
3 checks passed
@bougyman
bougyman deleted the issue-40 branch August 10, 2026 00:37
This was referenced Aug 11, 2026
bougyman pushed a commit that referenced this pull request Aug 11, 2026
🤖 I have created a release *beep* *boop*
---


##
[1.0.0](v1.0.0...v1.0.0)
(2026-08-11)


### ⚠ BREAKING CHANGES

* graduate to 1.0.0 - disable pre-major version bumping
([#70](#70))
* **cd:** parallelize Burrito target builds and fix Trivy/Podman image
scanning ([#68](#68))
* **ci:** rename release.yaml to main.yaml, workflow name to "main"
([#38](#38))
* add Readme/LICENSE, feat: wire up the issue list --project picker
([#12](#12))

### Features

* add mix githooks.install to activate the repo's git hooks
([#51](#51))
([9b35472](9b35472))
* **api:** add LinearCli.Api GraphQL client (Phase 1)
([723f3f6](723f3f6))
* **ci:** add a full SBOM - app deps, OTP/Elixir runtime, container OS
packages ([#54](#54))
([e296fdd](e296fdd))
* **cli:** add favorite teams/projects, filtering list views by them
([#61](#61))
([6c858d9](6c858d9))
* **cli:** add issue create/develop/pr/take/update write commands (Phase
6)
([1649618](1649618))
* **cli:** add profiles - default team/project stored in local SQLite
([#57](#57))
([554e336](554e336))
* **cli:** add project update - post a status update to a project
([#43](#43))
([30dc6dc](30dc6dc))
* **cli:** make version respect --output json
([#33](#33))
([b160aad](b160aad))
* **cli:** resolve bare issue numbers via active profile, favorited
teams, or a team prompt
([#65](#65))
([8b183da](8b183da))
* **cli:** support Ruby's short subcommand aliases
([#15](#15))
([9fca6b5](9fca6b5))
* initial commit with ash submodule
([c2ceafb](c2ceafb))
* **linear:** add Ash domain resources for
Issue/Project/Team/User/Label/WorkflowState/Comment (Phase 2)
([e2a27f3](e2a27f3))
* **oban:** add scheduled monthly project rollover (Phase 7)
([11afb43](11afb43))
* phase 4 from initial plan -&gt; complete
([012866e](012866e))
* phase 8 - packaging, releasing, and CI
([#1](#1))
([905c238](905c238))
* scaffold Elixir port and enforce conventional commits
([a4d03a0](a4d03a0))


### Bug Fixes

* **ci:** consolidate the release pipeline into one workflow/DAG
([ba821bb](ba821bb))
* **ci:** create releases as drafts so assets survive Immutable Releases
([#19](#19))
([8572623](8572623))
* **ci:** package release binaries with the wrapper scripts
([#24](#24))
([5fb24a4](5fb24a4))
* **ci:** rebuild the release pipeline to stop the version-bump runaway
loop ([#30](#30))
([c311fd4](c311fd4))
* **ci:** relabel the release PR as tagged after we tag it ourselves
([ab1408e](ab1408e))
* **ci:** skip commit-subject validation in the post-merge pipeline
([#41](#41))
([3d00a85](3d00a85))
* **cli:** reject unrecognized flags instead of treating them as issue
ids ([#2](#2))
([#4](#4))
([e697ff6](e697ff6))
* **deps:** update ash to a non-vulnerable version
([#63](#63))
([b838e2a](b838e2a))


### Performance Improvements

* **linear:** fan out find-by-ids and per-team project fetches (Phase 5)
([fb00153](fb00153))


### Documentation

* add Readme/LICENSE, feat: wire up the issue list --project picker
([#12](#12))
([eb42c69](eb42c69))


### Miscellaneous Chores

* **ci:** rename release.yaml to main.yaml, workflow name to "main"
([#38](#38))
([2581a40](2581a40))
* graduate to 1.0.0 - disable pre-major version bumping
([#70](#70))
([87a65ae](87a65ae))


### Continuous Integration

* **cd:** parallelize Burrito target builds and fix Trivy/Podman image
scanning ([#68](#68))
([5cc829a](5cc829a))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
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.

Validate Commit Subjects should not re-run in the post-merge pipeline

2 participants