fix: move test deps from PREREQ_PM to TEST_REQUIRES#40
Draft
toddr-bot wants to merge 1 commit into
Draft
Conversation
DBD::SQLite and Test::More were listed in PREREQ_PM, making them runtime dependencies. Anyone installing Tie::DBI via CPAN would pull in DBD::SQLite even when using MySQL/Postgres. Move them to TEST_REQUIRES (with fallback for EUMM < 6.64). Also update META_MERGE to META spec 2.0 structured format for repository, bugtracker, and license resources. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Move
DBD::SQLiteandTest::MorefromPREREQ_PM(runtime) toTEST_REQUIRES(test-only), and update META_MERGE to META spec 2.0 structured format.Why
PREREQ_PMdeclares runtime dependencies. Anyone installingTie::DBIvia CPAN getsDBD::SQLitepulled in even when they're using MySQL or Postgres — unnecessary download and compilation. These are test-only dependencies and belong inTEST_REQUIRES.The META resources were also using the legacy flat format, which prevents MetaCPAN from rendering proper repository/bugtracker links.
How
%test_requireshash for DRY reuseTEST_REQUIRESon EUMM >= 6.64, fall back toPREREQ_PMon older versionsrepository(type/url/web),bugtracker(web),license(arrayref)Testing
perl Makefile.PLgenerates correct MYMETA.json with test deps inprereqs.test.requiresmake testpassesmake disttestpasses🤖 Generated with Claude Code