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

Revert PR #12 #18

Closed
wants to merge 3 commits into from
Closed

Conversation

hakonhagland
Copy link
Contributor

Refer to #17 (comment)

Revert the temporary fix in PR #12 as the temporary fix may not be needed after Test::Vars has been updated.

Revert the temporary fix in PR perl-ide#12 as the temporary fix may
not be needed after Test::Vars has been updated.
Copy link
Member

@oalders oalders left a comment

Choose a reason for hiding this comment

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

This looks great. Can we also bump the minimum version of Test::Vars to the latest?

Bump minimum version of Test::Vars to 0.017
@hakonhagland
Copy link
Contributor Author

Can we also bump the minimum version of Test::Vars to the latest?

@oalders See last commit. Does it look right?

@@ -6,7 +6,7 @@ requires "Moo" => "0";
requires "PPI::Document" => "0";
requires "Path::Tiny" => "0";
requires "Pod::Usage" => "0";
requires "Test::Vars" => "0";
Copy link
Member

Choose a reason for hiding this comment

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

I think this line is enough. We don't need to set the version in dist.ini and the other changes will be generated by Dist::Zilla as well. I could have been a lot clearer about that. My bad!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No problem! I have reverted the other changes :)

@oalders
Copy link
Member

oalders commented Sep 16, 2024

Thank you so much @hakonhagland! I've squashed and merged at the command line via 78461f1

@oalders oalders closed this Sep 16, 2024
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