-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Adding docs for dev env, build and tests for contributing #5370
Conversation
76e3dad
to
e8538fc
Compare
CONTRIBUTING.md
Outdated
# Ways to contribute |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work.
Consider ripping off this https://github.com/dotnet/corefx/blob/master/CONTRIBUTING.md and refactoring the doc into 2 sections - contributing information (coding styles, source structure, making pull requests, working with the team); and a separate Developer guide (how to enlist, build, run tests)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
to be clear, I'm OK merging this to get started and we can have subsequent PRs to add more to the document. This is certainly much better than our current contributing.md 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok. I have updated it to have 2 main sections. Let me know what you think.
I am pointing to the docs we already have in the Wiki. Probably we would prefer to have all docs in the repo itself instead of the wiki (or generate some static pages with Jekyll or something into github-pages instead of the simple github wiki)
Let me know what you think.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
||
## Setting up a developer environment | ||
- Install Java Development Kit 8 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add a link
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I removed the link as per @JonathanGiles suggestion that we shouldn't tell people to use Oracle Java or OpenJDK , but just ask for JDK8.
Makes sense I think
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
e8538fc
to
b13e7b3
Compare
preview
![image](https://user-images.githubusercontent.com/24213737/64883923-4615c580-d615-11e9-921c-e3ac57820f0d.png)
![image](https://user-images.githubusercontent.com/24213737/64883935-4f069700-d615-11e9-8b16-cf74fcc2953d.png)