Skip to content

Gate release publishing with project checks#140

Merged
fabian-barney merged 3 commits into
mainfrom
codex/132-release-quality-gates
May 15, 2026
Merged

Gate release publishing with project checks#140
fabian-barney merged 3 commits into
mainfrom
codex/132-release-quality-gates

Conversation

@fabian-barney
Copy link
Copy Markdown
Owner

Summary

  • add the cognitive-java gate to the release workflow before publishing
  • build and resolve the CLI jar during releases
  • run the same Maven-source and Gradle-plugin crap-java gates used by CI before deployment steps

Closes #132

Verification

  • mvn verify
  • git diff --check

Copilot AI review requested due to automatic review settings May 15, 2026 18:12
Copy link
Copy Markdown

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

Adds the repo’s existing quality gates to the tag-based release workflow so releases can’t publish artifacts unless they pass the same cognitive-java and crap-java checks used in CI (closing #132).

Changes:

  • Run mvn cognitive-java:check in the release workflow before any publish/deploy steps.
  • Build and resolve the CLI shaded JAR during release, then run crap-java gates for Maven sources and the Gradle plugin sources.
  • Ensure the Gradle wrapper is executable for the Gradle-plugin crap-java gate step.

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

Comment thread .github/workflows/release.yml Outdated
Copy link
Copy Markdown

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread .github/workflows/release.yml
Copy link
Copy Markdown

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

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

@fabian-barney fabian-barney merged commit a74d77b into main May 15, 2026
11 checks passed
@fabian-barney fabian-barney deleted the codex/132-release-quality-gates branch May 15, 2026 18:37
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.

[Medium] code-quality: release.yml does not run cognitive/CRAP gates before publishing

2 participants