Skip to content

Due date and reviewer response#696

Merged
agitter merged 4 commits intogreenelab:masterfrom
agitter:response
Nov 17, 2017
Merged

Due date and reviewer response#696
agitter merged 4 commits intogreenelab:masterfrom
agitter:response

Conversation

@agitter
Copy link
Collaborator

@agitter agitter commented Nov 9, 2017

We should describe other changes we made since the initial submission in the response to reviewers.

@agitter
Copy link
Collaborator Author

agitter commented Nov 17, 2017

@dhimmel or @cgreene can you please look over this small pull request? I primarily want to put the revision due date in the readme.

@agitter agitter added this to the journal-revisions milestone Nov 17, 2017
@cgreene
Copy link
Member

cgreene commented Nov 17, 2017

LGTM 👍

@agitter agitter merged commit cba4294 into greenelab:master Nov 17, 2017
@agitter agitter deleted the response branch November 17, 2017 17:16
dhimmel pushed a commit that referenced this pull request Nov 17, 2017
This build is based on
cba4294.

This commit was created by the following Travis CI build and job:
https://travis-ci.org/greenelab/deep-review/builds/303637542
https://travis-ci.org/greenelab/deep-review/jobs/303637543

[ci skip]

The full commit message that triggered this build is copied below:

Due date and reviewer response (#696)

* Add due date to readme

* Note other changes in reviewer response

* Note citation tag requirement for forbidden characters

* Add back tag example
dhimmel pushed a commit that referenced this pull request Nov 17, 2017
This build is based on
cba4294.

This commit was created by the following Travis CI build and job:
https://travis-ci.org/greenelab/deep-review/builds/303637542
https://travis-ci.org/greenelab/deep-review/jobs/303637543

[ci skip]

The full commit message that triggered this build is copied below:

Due date and reviewer response (#696)

* Add due date to readme

* Note other changes in reviewer response

* Note citation tag requirement for forbidden characters

* Add back tag example
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.

2 participants