Skip to content

Fix a missing Future->done#974

Merged
richvdh merged 1 commit intodevelopfrom
rav/fix_upgrade_test
Oct 16, 2020
Merged

Fix a missing Future->done#974
richvdh merged 1 commit intodevelopfrom
rav/fix_upgrade_test

Conversation

@richvdh
Copy link
Member

@richvdh richvdh commented Oct 15, 2020

Seems to have broken in #972. What I don't understand is why it is still working in the CI.

Seems to have broken in #972
@richvdh richvdh requested a review from a team October 15, 2020 23:45
@richvdh
Copy link
Member Author

richvdh commented Oct 15, 2020

oh, it turns out this is allowed in Future 0.45: https://metacpan.org/source/PEVANS/Future-0.45/Changes#L5. If we're dropping support for earlier versions we need to note this in cpanfile.

@richvdh richvdh merged commit c27779f into develop Oct 16, 2020
@richvdh richvdh deleted the rav/fix_upgrade_test branch October 16, 2020 10:07
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