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

Pin to specific revisions in github until releases of gems are made. #6130

Merged
merged 1 commit into from
Nov 19, 2024

Conversation

cjcolvar
Copy link
Member

Turns out that active_fedora-datastreams didn't need a release or any code changes so I reverted to ~> 0.5.0. speedy_af got a release because it didn't actually depend on an active_fedora release. This leaves active_fedora as the only dependency without a tag or release so I pinned it to a specific git commit. I did this because I think it might take some time for community review of the Fedora 6 support in ActiveFedora and a release to be cut.

Copy link
Contributor

@Dananji Dananji left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good as far as I can tell 👍

@cjcolvar cjcolvar merged commit e8afc33 into develop Nov 19, 2024
2 checks passed
@cjcolvar cjcolvar deleted the more_deps branch November 19, 2024 21:27
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