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

Update branch master -> main #189

Merged
merged 1 commit into from
Sep 22, 2020
Merged

Conversation

dmcollins
Copy link
Contributor

GitHub Issue: (link)

  • Other Relevant Links (Google Groups discussion, related pull requests,
    Release pull requests, etc.)

What does this Pull Request do?

ansible-galaxy fails because some of the requirement branches have been renamed to 'main' and no longer have a 'master' branch. This fix consistently uses the renamed 'main' branches for everything.

How should this be tested?

Run ansible-galaxy install -r requirements.yml

Additional Notes:

Requirements that experience the failure are: activemw, cantaloupe, drupal-openseadragon, fcrepo, fcrepo-syn, fits, grok, karaf, and keymaster.

Interested parties

@Islandora-Devops/committers

Copy link
Contributor

@ruebot ruebot left a comment

Choose a reason for hiding this comment

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

Just hit this too, and just was about to put in a pull request.

@dannylamb
Copy link
Member

RTBC

@dannylamb dannylamb merged commit 2309762 into Islandora-Devops:dev Sep 22, 2020
@ruebot
Copy link
Contributor

ruebot commented Sep 22, 2020

@dannylamb should this same change be made on the main branch of this repo too? If so, I can do a PR real quick if you'd like.

@ruebot
Copy link
Contributor

ruebot commented Sep 22, 2020

oh, nvm. Versions are pinned there 🤦‍♂️

@dannylamb
Copy link
Member

we'll pull dev into main when we do our next release. it should be ok since it's pinned to specific versions.

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