Skip to content

docs: note that this repository's own pull requests are reviewed #142

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
jugonzalez12 wants to merge 1 commit into from
Closed

docs: note that this repository's own pull requests are reviewed #142

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,9 @@ Shared GitHub workflows and actions for ConductorOne connector repositories.

Runs Claude-powered PR review for same-repository pull requests. Fork PRs are skipped
because they do not receive automatic Claude review. Review jobs check out the exact PR
head SHA so Claude can inspect the proposed source tree locally. The action builds its
head SHA so Claude can inspect the proposed source tree locally. Pull requests to this
repository are reviewed as well, by the workflow and action on `main` rather than the
version a pull request proposes. The action builds its
Comment on lines +9 to +11

@github-actions github-actions Bot Oct 2, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Suggestion (confidence: medium): This sentence is in the section about the default connector profile, but only general-pr-review.yaml reviews this repository. pr-review.yaml:24 still skips ConductorOne/github-workflows on pull_request, and its run on this PR was skipped. Consider naming general-pr-review.yaml here, e.g. "Pull requests to this repository are reviewed by the general-pr-review.yaml workflow and action on main…".

prompt from a shared base prompt plus optional built-in mixins. The default profile is
`connector`, which adds the connector mixin for repos covered by connector required
workflows or rulesets.
Expand Down
Loading