Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request adds a new skill for staging the Flutter site. The instructions are clear overall, but I have a couple of suggestions to improve the clarity and logical flow of the steps. Specifically, I recommend reordering the staging steps to ensure validation happens on the latest content, and clarifying the usage of the two different link-checking commands.
|
Visit the preview URL for this PR (updated for commit bb9f62f): https://flutter-docs-prod--pr13166-staging-site-skill-0ay604fs.web.app |
There was a problem hiding this comment.
Maybe we should consider a different name and description for this skill? It slightly conflates the meaning of the word "staging": what the skill discusses and then its step 4, which is also staging the site. I wonder that could be a bit confusing.
Would this skill be better described as something like "Validate(or check?) your changes [to the site] [before committing]"?
3 participants
As it says