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

updated README #27349

Merged
merged 1 commit into from Oct 2, 2018

Conversation

Projects
None yet
4 participants
@tanmay21
Contributor

tanmay21 commented Oct 2, 2018

No description provided.

@andresgalante

This comment has been minimized.

Show comment
Hide comment
@andresgalante

andresgalante Oct 2, 2018

Member

@tanmay21 thanks a lot! I won't merge it since e I am not a native English speaker.

cc/ @patrickhlauke or @mdo

Member

andresgalante commented Oct 2, 2018

@tanmay21 thanks a lot! I won't merge it since e I am not a native English speaker.

cc/ @patrickhlauke or @mdo

@Johann-S Johann-S added docs v4 labels Oct 2, 2018

@patrickhlauke

This comment has been minimized.

Show comment
Hide comment
@patrickhlauke

patrickhlauke Oct 2, 2018

Member

Seems fairly non-controversial (as in we adhere now, rather than only in future). That whole paragraph is a horrible mouthful though...

Member

patrickhlauke commented Oct 2, 2018

Seems fairly non-controversial (as in we adhere now, rather than only in future). That whole paragraph is a horrible mouthful though...

@patrickhlauke patrickhlauke merged commit cd88061 into twbs:v4-dev Oct 2, 2018

3 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 88.62%
Details
deploy/netlify Deploy preview ready!
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment