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

Auto Pull REMARKs via GitHub Action #75

Merged
merged 5 commits into from
Oct 10, 2024
Merged

Conversation

camriddell
Copy link
Collaborator

@camriddell camriddell commented Aug 8, 2024

  • Remove Chinese-Growth.md
  • Remove Aiyagari-Idiosyncratic folder (shouldn't exist in this repo)
  • Remove all remaining REMARKs (to be included in a build step)
  • Add GitHub action to pull REMARKs before building

Copy link

netlify bot commented Aug 8, 2024

Deploy Preview for econark-website ready!

Name Link
🔨 Latest commit edbb67d
🔍 Latest deploy log https://app.netlify.com/sites/econark-website/deploys/66e2f84bf7bfc90008e8dbdf
😎 Deploy Preview https://deploy-preview-75--econark-website.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.

@llorracc
Copy link
Contributor

@camriddell, this PR is failing checks ...

@camriddell
Copy link
Collaborator Author

@DrDrij it seems like the build failure here may be unrelated to the changes I made? Ruby failed to install the dependencies to build the site.

@camriddell camriddell changed the title Removal of Stale/unused materials Auto Pull REMARKs via GitHub Action Sep 12, 2024
@camriddell
Copy link
Collaborator Author

@DrDrij it seems like the build failure here may be unrelated to the changes I made? Ruby failed to install the dependencies to build the site.

Saw your update on the Ruby version, just had to sync up :)

@camriddell
Copy link
Collaborator Author

@DrDrij with your adjustments to Netlify- this PR is looking good to me. Can you take a final pass and merge when ready?

@camriddell camriddell merged commit 1908bc4 into econ-ark:master Oct 10, 2024
4 checks passed
@camriddell
Copy link
Collaborator Author

Merged with @llorracc approval.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants