-
Notifications
You must be signed in to change notification settings - Fork 5
[Automation] Claude agents and workflows #955
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.
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
base: master
Are you sure you want to change the base?
Conversation
Added Topic Structure Agent documentation for auditing compliance with CTRT framework. Signed-off-by: Justine Geffen <[email protected]>
Signed-off-by: Justine Geffen <[email protected]>
Added voice-tone guidelines for documentation PRs, focusing on consistent voice, active tone, and present tense usage. Signed-off-by: Justine Geffen <[email protected]>
Signed-off-by: Justine Geffen <[email protected]>
Signed-off-by: Justine Geffen <[email protected]>
Signed-off-by: Justine Geffen <[email protected]>
Signed-off-by: Justine Geffen <[email protected]>
Signed-off-by: Justine Geffen <[email protected]>
Signed-off-by: Justine Geffen <[email protected]>
✅ Deploy Preview for seqera-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pre-commit formatting suggestions
Pre-commit hooks found some formatting changes. You can apply each suggestion directly by clicking the "Commit suggestion" button, or run pre-commit locally and commit again.
| ## Fixes Applied: [filename] | ||
|
|
||
| ✅ Line 23: Third person → Second person | ||
| ✅ Line 45: Product name standardization |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| ✅ Line 45: Product name standardization | |
| ✅ Line 45: Product name standardization |
|
|
||
| 1. Select **Launchpad** > **Add pipeline**. | ||
| 2. Enter the repository URL. | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| # Maximum files per review batch (prevents timeout on large PRs) | ||
| max_files_per_batch: 20 | ||
|
|
||
| # Output format for PR comments |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| # Output format for PR comments | |
| - major_change # Run when >50% of file changed | ||
|
|
||
| # File patterns (glob) | ||
| patterns: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| patterns: | |
| Use the terminology agent to review these changed documentation files: | ||
| ${{ needs.changes.outputs.docs_files }} | ||
|
|
||
| Check for: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| Check for: | |
| Use the clarity agent to review these changed documentation files: | ||
| ${{ needs.changes.outputs.docs_files }} | ||
|
|
||
| Check for: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| Check for: | |
| Use the inclusive-language agent to review these changed documentation files: | ||
| ${{ needs.changes.outputs.docs_files }} | ||
|
|
||
| Check for: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| Check for: | |
| const statusEmoji = { | ||
| 'success': '✅', | ||
| 'failure': '❌', | ||
| 'skipped': '⏭️', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| 'skipped': '⏭️', | |
| prompt: | | ||
| Use the docs-fix agent to apply fixes to all changed documentation files in this PR. | ||
|
|
||
| Apply fixes for: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| Apply fixes for: | |
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Signed-off-by: Justine Geffen <[email protected]>
…nto justine-claude
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pre-commit formatting suggestions
Pre-commit hooks found some formatting changes. You can apply each suggestion directly by clicking the "Commit suggestion" button, or run pre-commit locally and commit again.
| # Focus only on RNA-Seq issues | ||
| ./lint --no-heading-case platform-enterprise_docs/ | ||
|
|
||
| # Focus only on heading issues |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| # Focus only on heading issues | |
| # Focus only on heading issues |
| - Reason: [Google style guide principle or grammar rule] | ||
|
|
||
| + Line Y: [original text] | ||
| - Issue: [description of problem] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| - Issue: [description of problem] | |
| - Issue: [description of problem] |
| - Specific line-by-line corrections | ||
| - Suggestions for improvement | ||
| - Explanations of Google style guide principles | ||
| - A revised version if requested |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| - A revised version if requested | |
| - A revised version if requested |
|
|
||
| # Documentation Linter Shortcut | ||
| # Usage: ./lint [options] [directories...] | ||
| # |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| # | |
| # |
| else | ||
| # Pass all arguments through to the linter | ||
| node "$LINTER_PATH" "$@" | ||
| fi No newline at end of file |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| fi | |
| fi |
📝 Documentation Review Summary
Review powered by Claude Code SME agents To apply fixes: Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pre-commit formatting suggestions
Pre-commit hooks found some formatting changes. You can apply each suggestion directly by clicking the "Commit suggestion" button, or run pre-commit locally and commit again.
| else | ||
| # Pass all arguments through to the linter | ||
| node "$LINTER_PATH" "$@" | ||
| fi No newline at end of file |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| fi | |
| fi |
📝 Documentation Review Summary
Review powered by Claude Code SME agents To apply fixes: Comment |
Uh oh!
There was an error while loading. Please reload this page.