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

Dependency upgrades #152

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Dependency upgrades #152

wants to merge 4 commits into from

Conversation

ShanaLMoore
Copy link

@ShanaLMoore ShanaLMoore commented Aug 8, 2023

@cjcolvar
Copy link
Member

cjcolvar commented Aug 9, 2023

Hi @ShanaLMoore, I worked on this gem years ago and thought I'd try to help clear up the build issues that are blocking this PR. I refreshed the deploy key on the CircleCI side to fix the checkout issue and opened a PR to this PR that updates the container image being used for the build which gets it another step further. But it is still failing at building the test app through engine_cart. This is as much as I can work on it right now. Good luck and reach out if you get stuck.

@ShanaLMoore
Copy link
Author

ShanaLMoore commented Aug 9, 2023

Thanks for the message @cjcolvar I haven't spent much time digging into the build issues yet.

We are working on upgrading GBH to Hyrax 4 and needed to upgrade some dependencies for them. I have no idea if we broke anything yet but noticed that main is failing too 😓

@cjcolvar
Copy link
Member

cjcolvar commented Aug 9, 2023

No worries! I don't think it was anything y'all did. I'm pretty sure that because this repo hasn't been maintained it broke sometime due to dependencies or infrastructure changes.

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.

3 participants