Skip to content

feat: add apply-rulesets.sh to create pr-quality repository ruleset#65

Closed
claude[bot] wants to merge 1 commit intomainfrom
claude/issue-48-20260406-0342
Closed

feat: add apply-rulesets.sh to create pr-quality repository ruleset#65
claude[bot] wants to merge 1 commit intomainfrom
claude/issue-48-20260406-0342

Conversation

@claude
Copy link
Copy Markdown
Contributor

@claude claude Bot commented Apr 6, 2026

Summary

  • Adds scripts/apply-rulesets.sh which creates or updates the standard pr-quality repository ruleset on any petry-projects repo via the GitHub REST API
  • Implements all settings required by standards/github-settings.md#pr-quality

Ruleset configuration

Setting Value
Target branch Default branch (~DEFAULT_BRANCH)
Enforcement Active
Required approving reviews 1
Dismiss stale reviews on push Yes
Required review thread resolution Yes
Require code owner review Yes
Require last push approval Yes
Allow force pushes No
Allow deletions No
Squash-only merges Enforced via required_linear_history

Usage

After merging, run with an admin token to apply the ruleset to a repo:

# Apply to a specific repo
GH_TOKEN=<admin-token> ./scripts/apply-rulesets.sh .github

# Dry-run to preview changes
DRY_RUN=true GH_TOKEN=<admin-token> ./scripts/apply-rulesets.sh .github

# Apply to all org repos
GH_TOKEN=<admin-token> ./scripts/apply-rulesets.sh --all

Note on compliance closure

The compliance audit checks the live GitHub API for the pr-quality ruleset — merging this PR adds the tooling. Running the script with admin credentials is the final step to close the finding.

Closes #48

Generated with Claude Code

Adds scripts/apply-rulesets.sh which creates or updates the standard
pr-quality repository ruleset on any petry-projects repo via the GitHub
API, resolving the missing-pr-quality compliance finding.

Closes #48

Co-authored-by: don-petry <don-petry@users.noreply.github.com>
@claude claude Bot mentioned this pull request Apr 6, 2026
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Apr 6, 2026

@don-petry
Copy link
Copy Markdown
Contributor

Closing — superseded by #71 which implements code-quality rulesets with dynamic check detection. The pr-quality ruleset from this PR should be added to apply-rulesets.sh as a follow-up.

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.

Compliance: missing-pr-quality

1 participant