-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
feat: replace Tabs with PackageManagerTabs across pages [i18nIgnore] #12492
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
Conversation
✅ Deploy Preview for astro-docs-2 ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify project configuration. |
Hello! Thank you for opening your first PR to Astro’s Docs! 🎉 Here’s what will happen next:
|
Lunaria Status Overview🌑 This pull request will not trigger status changes. Learn moreLunaria automatically ignores changes on specific PRs by adding a ignored keyword in its title. Found: You can change this by either removing the keyword above from the PR's title, or modifying the Tracked FilesNote The notes below indicate what would happen if the pull request is merged when triggering status changes. Since a ignored keyword was found in the PR's title, the status changes indicated below won't be applied.
Warnings reference
|
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.
LGTM! Good job and welcome to Team Docs! 🙌
As discussed in the discussion #12448, the documentation currently doesn’t use
PackageManagerTabs
consistently.Description:
<Tabs>
component with<PackageManagerTabs>
across multiple pages.Affected pages:
develop-and-build.mdx
clever-cloud.mdx
install-and-setup.mdx
This ensures a consistent user experience across the documentation, making it easier for users to find commands for their preferred package manager.
I’m happy to make any changes if needed.