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

Automatically install Lefthook during setup #532

Merged
merged 2 commits into from
Sep 11, 2024
Merged

Conversation

No767
Copy link
Member

@No767 No767 commented Sep 11, 2024

Summary

To make life easier, instead of running the lefthook install command after you install the npm deps, we just do it inside of an script right after installation. This also prevents me from reminding folks to do that constantly. Kinda of a neat and handy trick tbh.

Types of changes

What types of changes does your code introduce to the UC Merced's ACM Chapter Website?
Put an x in the boxes that apply

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Update (Updates to README.md, the documentation, etc)
  • Other (if none of the other choices apply)

Checklist

Put an x in the boxes that apply

  • If code changes were made then they have been tested.
    • I have updated the documentation to reflect the changes. (if appropriate)
  • Lint, format, and unit test workflows pass with my new changes
  • This PR does not address a duplicate issue or PR

Copy link

netlify bot commented Sep 11, 2024

Deploy Preview for ucmacm ready!

Name Link
🔨 Latest commit 4564b6b
🔍 Latest deploy log https://app.netlify.com/sites/ucmacm/deploys/66e136c624009d000815f1c6
😎 Deploy Preview https://deploy-preview-532--ucmacm.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

sonarcloud bot commented Sep 11, 2024

Copy link

netlify bot commented Sep 11, 2024

Deploy Preview for ucmacm-docs failed.

Name Link
🔨 Latest commit 4564b6b
🔍 Latest deploy log https://app.netlify.com/sites/ucmacm-docs/deploys/66e136c6d2873b0008beca7a

@No767 No767 merged commit e9a2c3b into react-rewrite Sep 11, 2024
12 of 16 checks passed
@No767 No767 deleted the noelle/postprep branch September 11, 2024 06:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant