Merged
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the documentation site experience and copy, including a new responsive header navigation, plus small CLI feedback-link plumbing changes.
Changes:
- Add a custom responsive header/nav component to Starlight (desktop links + mobile hamburger) and supporting CSS.
- Update docs copy/examples (including introducing
gsalias usage) and refresh UI screenshots. - Switch feedback links from a GitHub Discussions URL to
gh.ioshort links and update related tests.
Show a summary per file
| File | Description |
|---|---|
| skills/gh-stack/SKILL.md | Updates CLI usage snippets to show flags before positional args. |
| README.md | Aligns command synopsis formatting and updates gh stack alias usage ordering. |
| docs/src/styles/custom.css | Adds responsive show/hide rules for the custom header links vs hamburger menu. |
| docs/src/content/docs/introduction/overview.md | Adjusts headings/copy and adds gh stack checkout mention. |
| docs/src/content/docs/index.mdx | Updates getting-started examples (adds alias step, uses gs). |
| docs/src/content/docs/guides/workflows.md | Updates workflow examples to use gs and clarifies step wording. |
| docs/src/content/docs/guides/ui.md | Swaps screenshot reference to the new merge box image. |
| docs/src/content/docs/faq.md | Minor copy improvement around opening PRs as a stack. |
| docs/src/content.config.ts | Extends docs schema to add a default “private preview” banner. |
| docs/src/components/CustomHeader.astro | New custom header component (nav links + hamburger dropdown + script). |
| docs/src/assets/screenshots/stack-merge-box.png | Adds updated screenshot used in docs. |
| docs/astro.config.mjs | Wires the custom header by overriding Starlight’s SocialIcons component. |
| cmd/submit.go | Uses the new feedbackURL in generated PR body footer. |
| cmd/submit_test.go | Updates test expectations to the new feedback URL constant. |
| cmd/feedback.go | Replaces the old discussions URL with feedbackURL / feedbackFormURL and updates browsing behavior. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comments suppressed due to low confidence (1)
cmd/feedback.go:43
- The success message always says "Opening feedback form in your browser..." even when
argsis empty and the command opensfeedbackURL(which may be a non-form landing page). Consider making the message match the actual target (or vary it based on whether a title was provided).
if err := b.Browse(targetURL); err != nil {
return err
}
cfg.Successf("Opening feedback form in your browser...")
- Files reviewed: 14/16 changed files
- Comments generated: 3
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
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.
CustomHeader.astrocomponent and supporting CSS, improving navigation on both desktop and mobile.