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

Provide 8/25 status update #90

Merged
merged 3 commits into from
Aug 25, 2016
Merged

Provide 8/25 status update #90

merged 3 commits into from
Aug 25, 2016

Conversation

cgreene
Copy link
Member

@cgreene cgreene commented Aug 25, 2016

Incorporate a discussion of work to date and provide a set of steps to get new participants up to speed.

@cgreene
Copy link
Member Author

cgreene commented Aug 25, 2016

By the way - codeclimate is angry that the list elements 2, 3, and 4 don't start at '1'. I'm not sure why the markdown linter is flagging that.

review.

In about a week, we plan to move into the phase where we start to vigorously
discuss this question for each area that the review will cover (currently also
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe instead of "discuss this question" it would be more clear to state something like: "discuss the overarching question" or "discuss the main question" or "discuss the guiding question".

@cgreene
Copy link
Member Author

cgreene commented Aug 25, 2016

@gwaygenomics tried to make that a bit more clear - success?

@gwaybio
Copy link
Contributor

gwaybio commented Aug 25, 2016

I looked into the markdownlint question a bit - looks like we would have to set Rule MD029 to ordered for this check to pass. I don't know why ordered lists and unordered lists are mutually exclusive to the program - but it appears to be a non-issue.

LGTM

@cgreene cgreene merged commit 6f793dd into master Aug 25, 2016
@cgreene cgreene deleted the status-update-readme branch August 25, 2016 14:12
@agitter
Copy link
Collaborator

agitter commented Aug 25, 2016

It's already merged, but I like the status update

dhimmel pushed a commit to dhimmel/deep-review that referenced this pull request Nov 3, 2017
dhimmel pushed a commit to dhimmel/deep-review that referenced this pull request Jan 11, 2018
By positioning the `body` element, we tell Hypothesis to use it (rather
than the document) as the container element for the adder. (See
hypothesis/client#487 for more information.)

Closes: greenelab#90, hypothesis/client#619
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.

3 participants