Address issues found by Deepsource

biocommons / eutils

simplified searching, fetching, and parsing records from NCBI using their E-utilities interface

Apache License 2.0

58 stars 25 forks source link

Closed reece closed 10 months ago

reece commented 4 years ago

Triage issues at https://deepsource.io/gh/biocommons/eutils/ and create a plan for resolving them. Options are:

richardNam commented 4 years ago

Hi @reece I can take a look at the second issue Undefined name detected. Anything I should be aware of before I dive in?

reece commented 4 years ago

I don't think there are any surprises here. Go for it! Please get in touch if you need help. Thanks for diving in, @richardNam!

reece commented 4 years ago

Comments:

pubmedcentralarticle.py: This couldn't be working. It's likely a copy-paste error. This also means that we need a test here. :-(
entrezgene.py: Also couldn't be working. There's supposed to be a class EntrezgeneLocus, but it's not imported... and, worse, it doesn't even exist apparently.
einforesult.py: Probably just needs an import in both cases.

github-actions[bot] commented 11 months ago

This issue is stale because it has been open 90 days with no activity. Remove stale label or comment or this will be closed in 7 days.

github-actions[bot] commented 10 months ago

This issue was closed because it has been stalled for 7 days with no activity.