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

Fix format for multiple citations in JOSS paper #87

Merged
merged 1 commit into from
Oct 30, 2019
Merged

Fix format for multiple citations in JOSS paper #87

merged 1 commit into from
Oct 30, 2019

Conversation

leouieda
Copy link
Contributor

@leouieda leouieda commented Oct 30, 2019

Multiple citations are separated with a ; instead of , which was
causing the paper to not render properly. Also fixes the missing
citation of Arribas-Bel2017 to arribas-bel2017looking which I assume
is what was intended (feel free to change if this is wrong).

Related to openjournals/joss-reviews#1838

Hello! Please make sure to check all these boxes before submitting a Pull Request
(PR). Once you have checked the boxes, feel free to remove all text except the
justification in point 5.

  1. You have run tests on this submission, either by using Travis Continuous Integration testing testing or running nosetests on your changes?
  2. This pull request is directed to the pysal/dev branch.
  3. This pull introduces new functionality covered by
    docstrings and
    unittests?
  4. You have assigned a
    reviewer
    and added relevant labels
  5. The justification for this PR is: Fixes JOSS reference formatting issues.

Multiple citations are separated with a `;` instead of `,` which was
causing the paper to not render properly. Also fixes the missing
citation of `Arribas-Bel2017` to `arribas-bel2017looking` which I assume
is what was intended (feel free to change if this is wrong).

Related to openjournals/joss-reviews#1838
@leouieda
Copy link
Contributor Author

Hi all, I'm not sure if this should go to the dev branch or not. It seems that the JOSS paper is in the master branch but let me know if I should change it.

@jGaboardi jGaboardi added JOSS paper priority: high needs to be resolved soon labels Oct 30, 2019
@jGaboardi jGaboardi added this to the JOSS paper milestone Oct 30, 2019
@slumnitz
Copy link
Member

Hi all, I'm not sure if this should go to the dev branch or not. It seems that the JOSS paper is in the master branch but let me know if I should change it.

Awesome, thank you for helping out! The master branch is the right place.

@slumnitz slumnitz closed this Oct 30, 2019
@slumnitz slumnitz reopened this Oct 30, 2019
@slumnitz slumnitz merged commit 011aba1 into pysal:master Oct 30, 2019
@leouieda leouieda deleted the joss-references branch October 31, 2019 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
JOSS paper priority: high needs to be resolved soon
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants