Skip to content

Remove artificial dependency on XML::Simple

Marek Szuba requested to merge no_xml_simple into master

Created by: mkszuba

Description

Remove the only line in the whole of the repository which (still?) depended on XML::Simple, and no longer require that module in cpanfile.

Use case

The bit of code in question, TAIROntologyParser, did not actually use that module - it merely imported XMLin from it and left it at that. Moreover, XML::Simple has long been deprecated.

Benefits

Gets rid of an unnecessary dependency.

Possible Drawbacks

None I can think of, considering the status of XML::Simple and that there is still at least one XML parser - namely XML::LibXML - we depend on.

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?

Run the entire test suite both before and after the change. It has passed on both occasions.

Merge request reports