Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Always report bibtex entries in DOI checker #101

Merged
merged 4 commits into from
Mar 6, 2024

Conversation

sneakers-the-rat
Copy link
Contributor

Not all bibliography entries have DOIs, and that's fine! But it can be confusing when there are bibliography entries and they don't show up in the DOI checker - what happened to them? It is semantically correct to say that DOIs are "missing" for entries where a DOI is not provided and one can't be found when querying crossref, and also for entries where a title is not provided, both of which are currently silently dropped by the DOI checker.

This PR adds else statements to the two unfinished legs of the check_dois method and updates the spec to match. Not sure if i wrote those test cases correctly, my bad if not.

@sneakers-the-rat
Copy link
Contributor Author

Sorry for the sloppy pull, someone is using my computer rn but I can fix once theyre done

Copy link
Member

@xuanxu xuanxu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed. Thanks @sneakers-the-rat!

@xuanxu xuanxu merged commit 946effa into openjournals:main Mar 6, 2024
5 checks passed
@arfon
Copy link
Member

arfon commented Mar 6, 2024

Thanks @sneakers-the-rat !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants