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

Incorporate new data from RECOVERY trial #864

Merged
merged 7 commits into from
Feb 18, 2021
Merged

Conversation

rando2
Copy link
Contributor

@rando2 rando2 commented Feb 16, 2021

Description of the proposed additions or changes

@Jeff-Field pointed out that the RECOVERY trial released data on TCZ last week - I did a quick write-up on this!
He also pointed out that we were using a verb that is not a verb, so I replaced all of those!

Related issues

Suggested reviewers (optional)

@SiminaB

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.

@rando2 rando2 added therapeutic pharmacology Text for Review Feedback on this text is welcome! labels Feb 16, 2021
@AppVeyorBot
Copy link

AppVeyor build 1.0.3308 for commit 65bb385 is now complete.

Found 2 potential spelling error(s). Preview:content/20.pharmaceuticals.md:146:Randomised
content/20.pharmaceuticals.md:664:administrtion...
The rendered manuscript from this build is temporarily available for download at:

@AppVeyorBot
Copy link

AppVeyor build 1.0.3309 for commit 618e738 is now complete.

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

Copy link
Collaborator

@SiminaB SiminaB left a comment

Choose a reason for hiding this comment

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

Generally looks very good! I only have a couple of clarification questions.

content/20.pharmaceuticals.md Outdated Show resolved Hide resolved
content/20.pharmaceuticals.md Outdated Show resolved Hide resolved
Copy link
Contributor Author

@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.

Thank you so much @SiminaB!

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.3318 for commit caa12d0 is now complete.

Found 1 potential spelling error(s). Preview:content/20.pharmaceuticals.md:566:overactivation...
The rendered manuscript from this build is temporarily available for download at:

@rando2
Copy link
Contributor Author

rando2 commented Feb 17, 2021

@SiminaB if these changes looked OK to you, would you mind approving so it can be merged? Thank you so much for the feedback!!!

@rando2 rando2 mentioned this pull request Feb 17, 2021
2 tasks
Copy link
Collaborator

@agitter agitter left a comment

Choose a reason for hiding this comment

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

I can approve this but found one typo first. Also have the two suggestions to address @SiminaB's comments been applied?

content/20.pharmaceuticals.md Outdated Show resolved Hide resolved
@rando2
Copy link
Contributor Author

rando2 commented Feb 18, 2021

Hi @agitter, I just committed everything! I assume it's usually easier for reviewers if I leave the responses active, but in this case it makes sense to commit them!

Copy link
Collaborator

@agitter agitter left a 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. Thanks for the TCZ updates.

@AppVeyorBot
Copy link

AppVeyor build 1.0.3340 for commit bfd8728 is now complete.

Found 1 potential spelling error(s). Preview:content/20.pharmaceuticals.md:867:monotherapy...
The rendered manuscript from this build is temporarily available for download at:

@rando2 rando2 merged commit 7d6b6be into greenelab:master Feb 18, 2021
@rando2 rando2 deleted the jf-edits branch February 18, 2021 19:29
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.

4 participants