-
Notifications
You must be signed in to change notification settings - Fork 42
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
Support raw citations #74
Conversation
98d0cd4
to
ceab6aa
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. See one optional comment.
Once this is merged, we should update the manubot-rootstock USAGE.md
(edit: I see you already noted this above).
manubot/process/util.py
Outdated
@@ -268,6 +270,12 @@ def generate_csl_items(args, citation_df): | |||
if citation in manual_refs: | |||
csl_items.append(manual_refs[citation]) | |||
continue | |||
elif citation.startswith('raw:'): | |||
logging.error( | |||
f'CSL JSON data with a standard_citation of {citation} not found in manual-references.json. ' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
data
has been capitalized in other instances of CSL JSON Data
.
Closes #62
TODO:
manubot cite
. Done, opened Cite command error handling #77 to create more succinct error messages heremanual-references.json