Skip to content

fix: disable wiki to comply with has_wiki standard - #186

Closed
don-petry wants to merge 28 commits into
mainfrom
claude/issue-165-20260414-1133
Closed

don-petry wants to merge 28 commits into
mainfrom
claude/issue-165-20260414-1133

Conversation

@don-petry

Copy link
Copy Markdown
Collaborator

Summary

  • Adds has_wiki: false to .github/settings.yml so the probot/settings app disables the wiki on this repository
  • Documentation lives in the repo; wiki should be off per org standards

Standard Reference

standards/github-settings.md#repository-settings--standard-defaults

Closes #165

Generated with Claude Code

Closes #165

Co-authored-by: don-petry <don-petry@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 14, 2026 11:33
@don-petry

Copy link
Copy Markdown
Collaborator Author

@don-petry — you're the code owner for this repo. This is a one-line settings fix to disable the wiki per org standards. Please review and merge when ready.

@coderabbitai

coderabbitai Bot commented Apr 14, 2026 •

Copy link
Copy Markdown

Warning

Rate limit exceeded

@github-actions[bot] has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 50 minutes and 17 seconds before requesting another review.

You’ve run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 28499e62-4804-4fe3-91dd-6febd2ca10b9

📥 Commits

Reviewing files that changed from the base of the PR and between 1f31eeb and 3efe057.

📒 Files selected for processing (1)
  • .github/settings.yml
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/issue-165-20260414-1133

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR brings the repository into org compliance by explicitly disabling the GitHub wiki via probot/settings configuration.

Changes:

  • Adds repository.has_wiki: false to .github/settings.yml to ensure the wiki is disabled per the referenced standard.

@don-petry

don-petry commented Apr 16, 2026 •

Copy link
Copy Markdown
Collaborator Author
Superseded by automated re-review at 4feb986a8a4b67d3125f6938907aba727957ec23 — click to expand prior review.

Automated review — APPROVED

Risk: LOW
Reviewed commit: 8984228911e16abf83c79595e3e533ac1ff85ba5
Cascade: triage → deep (see triage: haiku 4.5 → deep: sonnet 4.6 + duck: gpt-5.4 → audit: opus 4.6 for models)

Summary

Single-line YAML config change adds has_wiki: false to .github/settings.yml so probot/settings disables the wiki per org standards. All CI checks pass (CodeQL, SonarCloud, dependency audit, Node tests, Playwright), zero new issues. No code logic, no secrets, no security surface — straightforward org-compliance fix.

Findings

Info

  • .github/settings.yml:4 — Change correctly applies org standard has_wiki: false under repository: key; probot/settings YAML structure is valid.

CI status

All CI checks pass (CodeQL, SonarCloud, dependency audit, Node tests, Playwright).


Reviewed by the don-petry PR-review cascade (triage: haiku 4.5 → deep: sonnet 4.6 + duck: gpt-5.4 → audit: opus 4.6). Reply with `@don-petry` if you need a human.

@don-petry
don-petry enabled auto-merge (squash) April 16, 2026 20:43

@petry-projects-pr-review-agent petry-projects-pr-review-agent Bot 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.

Reviewed by the don-petry PR-review cascade (triage: haiku 4.5 → deep: sonnet 4.6 + duck: gpt-5.4 → audit: opus 4.6). Reply with `@don-petry` if you need a human.

@petry-projects-pr-review-agent petry-projects-pr-review-agent Bot 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.

Reviewed by the don-petry PR-review cascade (triage: haiku 4.5 → deep: sonnet 4.6 + duck: gpt-5.4 → audit: opus 4.6). Reply with `@don-petry` if you need a human.

@github-actions
github-actions Bot requested a review from a team as a code owner May 4, 2026 20:39

@donpetry-bot donpetry-bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Reviewed by the don-petry PR-review cascade (triage: haiku 4.5 → deep: sonnet 4.6 + duck: gpt-5.4 → audit: opus 4.6). Reply with `@don-petry` if you need a human.

@don-petry don-petry closed this May 11, 2026
auto-merge was automatically disabled May 11, 2026 21:40

Pull request was closed

@don-petry don-petry reopened this May 11, 2026
@sonarqubecloud

Copy link
Copy Markdown

donpetry-bot
donpetry-bot previously approved these changes May 17, 2026

@donpetry-bot donpetry-bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Automated review — APPROVED ✓

Risk: LOW
Reviewed commit: 4feb986a8a4b67d3125f6938907aba727957ec23
Review mode: triage-approved (single reviewer)

Summary

Five-line addition to .github/settings.yml introduces a repository: block setting has_wiki: false so the probot/settings app disables the wiki per the org github-settings.md standard. Pure configuration change — no application code, no secrets, no dependency or workflow changes. Triage classified this as low-risk; that assessment is confirmed.

Linked issue analysis

Closes #165, which tracks bringing this repo into compliance with the org has_wiki: false default. The added repository.has_wiki: false directly satisfies the requirement, and the PR body links the standards doc (standards/github-settings.md#repository-settings--standard-defaults).

Findings

Info

  • .github/settings.yml:4-6 — repository.has_wiki: false is the correct probot/settings schema key under the top-level repository: map; placement above the existing labels: block is consistent with the file's section comments.

No blocking issues. No carryover items from prior review cycle (prior diff is just dependabot merges from main).

CI status

All required checks green at the reviewed SHA: CodeQL (actions, javascript-typescript, python), SonarCloud quality gate (0 new issues, 0 hotspots), CodeRabbit. mergeable: MERGEABLE (mergeStateStatus BLOCKED is the standard branch-protection gate awaiting this approval, not a CI failure).


Reviewed automatically by the PR-review agent (single-reviewer mode: opus 4.7). Reply if you need a human review.

@github-actions

Copy link
Copy Markdown
Contributor

Auto-rebase failed — merge conflict — this branch has conflicts with main that must be resolved manually.

Please resolve the conflicts and push:

git fetch origin
git merge origin/main
# resolve conflicts, then:
git add .
git commit
git push

@donpetry-bot donpetry-bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Automated review — APPROVED ✓

Risk: LOW
Reviewed commit: 3efe057179a5f3f9c31eb4f96001593f5d3a42cd
Review mode: triage-approved (single reviewer)

Summary

Five-line addition to .github/settings.yml introduces a repository: block setting has_wiki: false so the probot/settings app disables the wiki per the org github-settings.md standard. Pure configuration change — no application code, no secrets, no dependency or workflow changes. Triage classified this as low-risk; that assessment is confirmed, and the diff is unchanged from the prior approved review at 4feb986 (intervening commits are merge-main fast-forwards only).

Linked issue analysis

Closes #165, which tracks bringing this repo into compliance with the org has_wiki: false default. The added repository.has_wiki: false directly satisfies the requirement, and the PR body links the standards doc (standards/github-settings.md#repository-settings--standard-defaults).

Findings

Info

  • .github/settings.yml:4-6 — repository.has_wiki: false is the correct probot/settings schema key under the top-level repository: map; placement above the existing labels: block is consistent with the file's section-comment style.
  • The PR is currently in a CONFLICTING / DIRTY merge state and an auto-rebase comment from github-actions flagged a conflict with main that the author needs to resolve manually before merge. This is a merge-state issue, not a code-review blocker — approval stands once conflicts are resolved.

No blocking issues. No carryover items from prior review cycle.

CI status

All required checks green at the reviewed SHA: CodeQL (actions, javascript-typescript, python), CodeRabbit. SonarCloud previously passed with 0 new issues and 0 hotspots.


Reviewed automatically by the PR-review agent (single-reviewer mode: opus 4.7). Reply if you need a human review.

@donpetry-bot
donpetry-bot dismissed their stale review May 30, 2026 20:09

Superseded by automated re-review at 3efe057.

@donpetry-bot

Copy link
Copy Markdown
Contributor

Automated review — human attention needed

This PR has been through 3 automated review cycles (cap: 3) without converging on an approval-and-merge state. Further automated review has been paused to avoid infinite loops.

Please take a look manually, or close this PR if it's no longer needed. Once a human review resolves the situation, remove the needs-human-review label and the cascade can be re-engaged on the next push.

Posted by the donpetry-bot PR-review cascade.

@don-petry

Copy link
Copy Markdown
Collaborator Author

Closing due to merge conflict that cannot be auto-rebased. Re-implementing from fresh main via dev-lead.

@don-petry don-petry closed this Jun 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Compliance: has_wiki

3 participants