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

Force install Module::Pluggable so CI can run #1719

Merged
merged 1 commit into from
Nov 25, 2024

Conversation

cromedome
Copy link
Contributor

GitHub Actions, regardless of what we tell it, still insists on installing the incorrect version of Module::Pluggable. Instead, we'll hit it with a really big hammer and force it to install.

This can be removed once the author fixes the issue installing Module::Pluggable as root.

I don't like doing this but I also don't like holding up a release because tests aren't passing.

GitHub Actions, regardless of what we tell it, still insists on
installing the incorrect version of Module::Pluggable. Instead, we'll
hit it with a really big hammer and force it to install.

This can be removed once the author fixes the issue installing
Module::Pluggable as root.
Copy link
Member

@veryrusty veryrusty left a comment

Choose a reason for hiding this comment

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

Unfortunately this is the module we are looking for. 👍 for using the force.

cromedome pushed a commit that referenced this pull request Nov 25, 2024
    [ BUG FIXES ]
    * None

    [ ENHANCEMENTS ]
    * None

    [ DOCUMENTATION ]
    * None

    [ DEPRECATED ]
    * None

    [ MISC ]
    * PR #1715: Update deprecated download-artifact (dependabot)
    * GH #1716: Remove Scope::Upper from list of recommended deps
      (Russell @veryrusty Jenkins)
    * PR #1718: Add Module::Pluggable as a requirement, excluding
      broken versions (Russell @veryrusty Jenkins)
    * PR #1719: Force install of Module::Pluggable so CI can successfully
      run (Jason A. Crome, Russell @veryrusty Jenkins)
@cromedome cromedome merged commit e73d7b3 into main Nov 25, 2024
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants