Closed risharde closed 1 year ago
This is good. I made a few cleanups, some minor corrections, and fixed the issues Lacey raised. I also did a full test loading a GFF3 and all looks good. It still needs the functional testing to make sure all is good, but I think this can be merged once all the CI tests finish.
Code Climate has analyzed commit c0c9d7a7 and detected 30 issues on this pull request.
Here's the issue category breakdown:
Category | Count |
---|---|
Complexity | 30 |
The test coverage on the diff in this pull request is 0.0% (50% is the threshold).
This pull request will bring the total coverage in the repository to 20.4% (-3.0% change).
View more on Code Climate.
Checks passed! Merging
This is the working version of GFF3 Importer compatible with Drupal 9. Old deprecated functions for logging and using ChadoRecord calls now use the Tripal DB functions Removed unnecessary comments, print_r statements etc.
Tested on Citrus sinensis and Arabidopsis thaliana without errors on console. @spficklin also helped me verify that data was being added to the necessary tables.
TODO: Unit testing code (branching out from this branch).