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

Make Zenodo metadata file valid JSON #33

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

martintoreilly
Copy link

Should fix #32.

Unescapes single quote in description field of .zenodo.json file to make file pass JSON format validation. See issue #32 for details of problem and fix.

Also adds PyCharm IDE .idea folder to .gitignore.

Single quotes should not be escaped in JSON and were causing a parse error on Zenodo.
@martintoreilly martintoreilly changed the title 32 fix zenodo json Make Zenodo metadata file valid JSON Sep 14, 2018
@martintoreilly
Copy link
Author

@KirstieJane Could you take a look at this and merge if you are persuaded it might fix the Zenodo synchronisation issue? If it does, I'm not sure whether our best course of action is to tag the CW18 commit as v0.2 (as it seems to have been manually created already in Zenodo) or just to tag the RSE18 commit as v0.3.

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.

New Github releases not triggering new Zenodo version
1 participant