Skip to content

Move dump_test_schema.pl dependencies from a README to a cpanfile feature

Marek Szuba requested to merge feature/cpanfile_dump_test_schema_deps into master

Created by: mkszuba

Description

Make it easier to install dependencies of dump_test_schema.pl by having them declared in the cpanfile - in a separate feature because they introduce quite a few higher-order dependencies yet are only useful to people who patch test databases in Ensembl repositories.

Use case

Until now said dependencies were listed in scripts/README.dump_test_schema. This made it necessary for them to be installed by hand, moreover I suspect quite a few users didn't even know such a README file existed because it was stashed away in a subdirectory. Last but not least, the README seems slightly out of date.

Benefits

Simplify installation of dependencies.

Possible Drawbacks

Something new to learn to for people who have never worked with cpanfile features.

Testing

Have you added/modified unit tests to test the changes?

No.

If so, do the tests pass/fail?

N/A

Have you run the entire test suite and no regression was detected?

Yes, no regression detected.

Merge request reports