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

Editorial tweaks to the JOSS paper #35

Merged
merged 4 commits into from Jun 11, 2021
Merged

Editorial tweaks to the JOSS paper #35

merged 4 commits into from Jun 11, 2021

Conversation

leouieda
Copy link
Contributor

@leouieda leouieda commented Jun 11, 2021

Some very minor tweaks to the paper:

  • Make the URLs in the table actual links
  • Consistency in use of quotes around the package name
  • Missing comma

openjournals/joss-reviews#2766

Placing the URLs doesn't render them as links in the paper.
Use `[]()` to make them clickable.
@leouieda
Copy link
Contributor Author

Waiting on a preview of the PDF here: http://whedon.theoj.org/preview?id=5a684146d65de8c1ab4b34a3 Whedon seems to be taking a bit long.

@znicholls
Copy link
Contributor

Looks great, let me know when you're done and I'll merge

@leouieda
Copy link
Contributor Author

@znicholls I had some trouble with the rendering of the table so I simplified the Markdown syntax to fix them. The table looks slightly different in the rendered document but nothing major.

Here is a preview: paper.pdf

I'm done with this so feel free to make changes or merge.

@znicholls
Copy link
Contributor

Awesome thanks!

@znicholls znicholls merged commit dba809d into openscm:paper-typos Jun 11, 2021
@lewisjared
Copy link

Thanks @leouieda

@leouieda leouieda deleted the patch-1 branch June 14, 2021 14:27
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.

None yet

3 participants