Skip to content
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

Add Prettier pre-push hook #9283

Merged
merged 6 commits into from
Oct 29, 2024
Merged

Add Prettier pre-push hook #9283

merged 6 commits into from
Oct 29, 2024

Conversation

jazzsequence
Copy link
Contributor

@jazzsequence jazzsequence commented Oct 25, 2024

This pull request adds a Prettier pre-push hook to automatically format JavaScript files before pushing. It also adds a new section to the readme file explaining how to use the git hooks that have been added to the .github/hooks folder.

The .github/hooks folder includes a pre-push hook that runs Prettier to check for code styling issues and automatically adds the changes back to the branch if any are found.

The documentation explains how to use this pre-push hook since it is not possible to commit git hooks stored in the normal .git/hooks directory to a repository.

@jazzsequence jazzsequence requested a review from a team as a code owner October 25, 2024 21:01
@jazzsequence jazzsequence self-assigned this Oct 25, 2024
@jazzsequence jazzsequence added the Site - CI Continuous Integration work specific to the docs repo/site itself label Oct 25, 2024
Copy link

⚡ Deployed with Pantheon Decoupled

This build was successfully deployed with Pantheon. You can track the build logs here.

👀 Preview: https://pr-9283-documentation.appa.pantheon.site
🛠️ Manage in Pantheon: https://dashboard.pantheon.io/site/2b30153f-e8b1-4427-b076-6109e704ba5d/overview

Copy link

⚡ Deployed with Pantheon Decoupled

This build was successfully deployed with Pantheon. You can track the build logs here.

👀 Preview: https://pr-9283-documentation.appa.pantheon.site
🛠️ Manage in Pantheon: https://dashboard.pantheon.io/site/2b30153f-e8b1-4427-b076-6109e704ba5d/overview

@rachelwhitton rachelwhitton merged commit 265cef1 into main Oct 29, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Site - CI Continuous Integration work specific to the docs repo/site itself
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants