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

Pharmaceuticals proofreading #854

Merged
merged 11 commits into from
Feb 12, 2021

Conversation

agitter
Copy link
Collaborator

@agitter agitter commented Feb 11, 2021

Description of the proposed additions or changes

This is my proofreading of the pharmaceuticals section. It's marked as a draft because I still plan to close #851 in this pull request and review the other references. However, I wanted to push what I have in case it conflicts with #853.

Related issues

Closes #839
Closes #851
#738

Suggested reviewers (optional)

@rando2

Checklist

  • Text is formatted so that each sentence is on its own line.
  • Pre-prints cited in this pull request have a GitHub issue opened so that they can be reviewed.

@AppVeyorBot
Copy link

AppVeyor build 1.0.3156 for commit 5c22a32 failed.

@agitter
Copy link
Collaborator Author

agitter commented Feb 11, 2021

AppVeyor is failing due to the Docker Hub pull rate limit. Nothing we can do about that.

Copy link
Contributor

@rando2 rando2 left a comment

Choose a reason for hiding this comment

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

I will keep an eye out for additional updates, but here are two things I noticed! Thanks for catching all of these issues!!!

content/20.pharmaceuticals.md Outdated Show resolved Hide resolved
content/20.pharmaceuticals.md Outdated Show resolved Hide resolved
@AppVeyorBot
Copy link

AppVeyor build 1.0.3158 for commit 3957efa is now complete....

Found 0 potential spelling error(s). Preview: The rendered manuscript from this build is temporarily available for download at:

@AppVeyorBot
Copy link

AppVeyor build 1.0.3161 for commit c18712b is now complete.

Found 0 potential spelling error(s). Preview:...
The rendered manuscript from this build is temporarily available for download at:

@AppVeyorBot
Copy link

AppVeyor build 1.0.3164 for commit 3a953cd is now complete.

Found 0 potential spelling error(s). Preview:...
The rendered manuscript from this build is temporarily available for download at:

@AppVeyorBot
Copy link

AppVeyor build 1.0.3165 for commit 3625cee is now complete.

Found 0 potential spelling error(s). Preview:...
The rendered manuscript from this build is temporarily available for download at:

@AppVeyorBot
Copy link

AppVeyor build 1.0.3166 for commit dd58dc6 is now complete.

Found 0 potential spelling error(s). Preview:...
The rendered manuscript from this build is temporarily available for download at:

@agitter agitter marked this pull request as ready for review February 12, 2021 04:05
@agitter
Copy link
Collaborator Author

agitter commented Feb 12, 2021

This is ready for review.

@rando2 rando2 mentioned this pull request Feb 12, 2021
2 tasks
Copy link
Contributor

@rando2 rando2 left a comment

Choose a reason for hiding this comment

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

Screen Shot 2021-02-12 at 10 53 16 AM

Looks like it worked!

Copy link
Collaborator

@cbrueffer cbrueffer left a comment

Choose a reason for hiding this comment

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

Looks great, just a few minor suggestions.

content/20.pharmaceuticals.md Outdated Show resolved Hide resolved
content/20.pharmaceuticals.md Outdated Show resolved Hide resolved
Co-authored-by: Christian Brueffer <[email protected]>
@AppVeyorBot
Copy link

AppVeyor build 1.0.3171 for commit 5db4e7a is now complete.

Found 0 potential spelling error(s). Preview:...
The rendered manuscript from this build is temporarily available for download at:

@rando2 rando2 merged commit b615abb into greenelab:master Feb 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Broken references in pharmaceuticals Customizing front matter with individual manuscripts
4 participants