Skip to content

ci: remove the PR title lint workflow - #32

Merged
dannysteenman merged 1 commit into
mainfrom
ci/remove-pr-title-lint
Oct 13, 2026
Merged

dannysteenman merged 1 commit into
mainfrom
ci/remove-pr-title-lint

Conversation

@dannysteenman

Copy link
Copy Markdown
Member

Pull requests no longer get a "Validate PR title" check. The workflow was generated by projen, so this disables it at the source with github_options={"pull_request_lint": False} in .projenrc.py. Regenerating deleted .github/workflows/pull-request-lint.yml and its entries in .gitattributes, .gitignore and .projen/files.json.

The lint was the only projen-managed workflow that used a pinned third-party action. So this also removes the unused amannn/action-semantic-pull-request pin and the pin_github_actions helper from src/bin/cicd_helper.py. After regenerating, cdk-validate.yml and the deploy workflows are unchanged.

main has no branch protection, so no required status check depends on the lint. The README and docs never mentioned it.

Validation

  • uv run projen regenerates cleanly.
  • pytest passes (4 tests).
  • agent-validate check --pr passes.
  • ruff check reports no issues in the changed files. Repo-wide ruff check and ruff format --check still fail on files this PR doesn't touch, such as src/app.py and src/stacks/github_oidc_stack.py.

@dannysteenman dannysteenman added the impact:limited Two-way door: a revert fully undoes it, including anything already deployed or published label Oct 13, 2026
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-13T18:28:35.688683Z e603d84 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@dannysteenman
dannysteenman merged commit ee404ac into main Oct 13, 2026
3 checks passed
@dannysteenman
dannysteenman deleted the ci/remove-pr-title-lint branch October 13, 2026 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

impact:limited Two-way door: a revert fully undoes it, including anything already deployed or published

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant