Update defaults.py - Githubissues

ISA-tools / isa-api

ISA tools API

https://isa-tools.org

Other

40 stars 37 forks source link

Update defaults.py #539

Closed ptth222 closed 4 months ago

ptth222 commented 4 months ago

The previous regex didn't match any DOI I tried. I found this one in a blog post from Crossref where they say it matched 74.4M out of 74.9M DOIs that they have seen. The post is here: https://www.crossref.org/blog/dois-and-matching-regular-expressions/

ptth222 commented 4 months ago

I rebased to issue-511. It has the same changes that are on #549, so if that is merged first there should only be the changes to the DOI regex and 1 test.

coveralls commented 4 months ago

coverage: 81.299%. remained the same when pulling 2275ee912949fc20f1d653c282842ccfb2d272a0 on ptth222:doi-regex-fix into 8aae210613b4b1e1f69698b944faf091c81e05a2 on ISA-tools:issue-511.