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

Created check_site_changes.sh script #229

Merged
merged 4 commits into from
Nov 16, 2021
Merged

Created check_site_changes.sh script #229

merged 4 commits into from
Nov 16, 2021

Conversation

EthanThatOneKid
Copy link
Owner

@EthanThatOneKid EthanThatOneKid commented Nov 11, 2021

Created this script to check for any unmerged code that would affect the website output. If there are relevant changes, Vercel will be asked to build.

image

Closing #198 with a new script.

@vercel
Copy link

vercel bot commented Nov 11, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/ethanthatonekid/acm-csuf-site/5QQA1Ypfx46ECG5FVP62c3n8vjAY
✅ Preview: Failed

[Deployment for 4442ca2 canceled]

@vercel vercel bot temporarily deployed to Preview November 11, 2021 08:36 Inactive
@vercel vercel bot temporarily deployed to Preview November 12, 2021 01:05 Inactive
@EthanThatOneKid EthanThatOneKid changed the title Created check_site_changes.ts script Created check_site_changes.sh script Nov 12, 2021
@EthanThatOneKid EthanThatOneKid linked an issue Nov 12, 2021 that may be closed by this pull request
@diamondburned
Copy link
Collaborator

Has this been tested?

@EthanThatOneKid
Copy link
Owner Author

EthanThatOneKid commented Nov 12, 2021

Has this been tested?

It works as intended in #229.

Since the only change in this PR is a non-website change, Vercel skips building this PR. I tested the script locally to see if it exits correctly when there are changes though and both cases work correctly.

@EthanThatOneKid EthanThatOneKid merged commit f615306 into main Nov 16, 2021
@EthanThatOneKid EthanThatOneKid added bugfix A PR that fixes a bug devops Some operation must be executed labels Dec 8, 2021
@jaasonw jaasonw deleted the fix/198 branch February 21, 2022 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix A PR that fixes a bug devops Some operation must be executed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Only build when website files are updated
2 participants