Revise SKILL.md for updated documentation process#1399
Open
justinegeffen wants to merge 2 commits intomasterfrom
Open
Revise SKILL.md for updated documentation process#1399justinegeffen wants to merge 2 commits intomasterfrom
justinegeffen wants to merge 2 commits intomasterfrom
Conversation
Updated the skill documentation to reflect changes in input sources and workflow phases. Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
✅ Deploy Preview for seqera-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Member
|
fix formatting |
christopher-hakkaart
approved these changes
May 7, 2026
Member
christopher-hakkaart
left a comment
christopher-hakkaart
left a comment
There was a problem hiding this comment.
Looks good. I'd be tempted to make the scope narrower (specifically call out master branch and make it the docs repo unless otherwise specified. Just to keep things simple and clear, the more options, the more chance things could go wrong.
|
|
||
| Also read `references/docusaurus-conventions.md` in this skill's directory for Seqera-specific formatting patterns (admonitions, link references, frontmatter, heading hierarchy). | ||
|
|
||
| **Confirm the PR target repo before doing anything else.** Ask the user which repo the eventual PR should target. Default to `seqeralabs/docs`, but offer alternatives like `seqeralabs/platform-enterprise_docs`, a fork, or a feature-specific repo. Hold onto the answer — you'll need it in Phase 6. Also confirm the base branch (default: `master` or `main`, whichever the repo uses). |
Member
There was a problem hiding this comment.
Should this just be seqeralabs/docs with the option of a manual input custom repo?
IMO, it seems like an unnecessary step to list other options when they will be low-usage.
Member
There was a problem hiding this comment.
Do feel free to ignore, comment only
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Updated the skill documentation to reflect changes in input sources and workflow phases.