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

Return to v5.8, since this is a prerequisite to HTML::HeadParser and LWP::UserAgent #14

Open
wants to merge 7 commits into
base: dev
Choose a base branch
from

Conversation

briandfoy
Copy link

I don't see any v5.10 features here, and the tests pass under v5.8. This module is a pre-requisite to other modules that have minimum versions of v5.8, which means this module moving to v5.10 inadvertently breaks modules that should work with v5.8.

Also, along the way GitHub has screwed up workflows such that we can't use the old perl images: Perl/docker-per#161 . I threw in my basic Ubuntu workflow file from briandfoy/github_workflows, but you don't have to use that. It does have one change to get around installing this module from CPAN on v5.8.

The Changes files noted a lack of infrastructure to test on v5.8, but GitHub workflows and CPAN Testers will handle that. And, there are v5.8 docker images too.

We can't install from CPAN so we need to get there
first with a version that will work.
briandfoy added a commit to briandfoy/text-multimarkdown that referenced this pull request Sep 20, 2024
briandfoy added a commit to briandfoy/github_workflows that referenced this pull request Sep 20, 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.

1 participant