docs(workflows): clarify intentional double .github/ in reusable-workflow paths#139
docs(workflows): clarify intentional double .github/ in reusable-workflow paths#139
Conversation
…flow paths
The compliance checker (`reusable-workflow-path-duplicate-github`) flags
`petry-projects/.github/.github/workflows/` as a duplicate path segment,
but this is a false positive.
GitHub reusable-workflow syntax is `{owner}/{repo}/{path}@{ref}` where
`{path}` is relative to the repository root. The org's reusable workflows
live at `.github/workflows/` inside the `petry-projects/.github` repo, so
the "double `.github/`" is unavoidable and correct per ci-standards.md
§ Tier 1 stubs. Every other org repo (e.g. TalkTerm) uses the same path.
Add inline comments to both caller stubs to make the intent explicit and
prevent the compliance audit from re-flagging this as an error.
Closes #136
Co-authored-by: don-petry <don-petry@users.noreply.github.com>
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 33 minutes and 48 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the 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 configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
|
CODEOWNERS review request: @don-petry — this PR addresses the weekly compliance finding for issue #136. The change is documentation-only (inline comments clarifying why the |
|
|
Closing: duplicate .github/ is REQUIRED in reusable workflow paths. The format owner/repo/.github/workflows/ is correct. |
1 participant
Summary
# NOTE:comments toagent-shield.ymlanddependabot-automerge.ymlexplaining why thepetry-projects/.github/.github/workflows/path is correct and intentional.reusable-workflow-path-duplicate-githubis a false positive: the "double.github/" is unavoidable valid GitHub syntax because reusable-workflow paths follow{owner}/{repo}/{path}@{ref}, and the org's reusables are stored at.github/workflows/inside thepetry-projects/.githubrepo.uses:lines are unchanged. Every other org repo (e.g. TalkTerm) and the canonical templates instandards/workflows/use the same path.Root cause
The automated compliance checker identifies any
uses:path containing two.github/segments as a duplicate. For the special.githubrepository this pattern is structurally correct and cannot be avoided without breaking the workflow.Test plan
Closes #136
Generated with Claude Code