-
Notifications
You must be signed in to change notification settings - Fork 8
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
Update README.md and CONTRIBUTING.md #1081
Conversation
- Add brief summary and add URLS pointing to doc site.
* [PASS UI](https://github.com/eclipse-pass/pass-ui) | ||
* [PASS Support](https://github.com/eclipse-pass/pass-support) | ||
* [PASS Acceptance Testing](https://github.com/eclipse-pass) | ||
* [PASS Docker](https://github.com/eclipse-pass/pass-docker) |
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.
Should we include the PASS Documentation repo too?
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.
Done.
## Purpose | ||
* [Eclipse PASS Website](https://eclipse-pass.org/index.html) | ||
* [PASS Documentation](https://docs.eclipse-pass.org/) | ||
* [PASS JHU User Guide](https://pass.jhu.edu/guide/) |
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'm wondering if we should include a link to the CONTRIBUTING.md file here?
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.
The contributing file directly links to the contributing page on docs.eclipse-pass.org. Perhaps we just directly link to that and save the user a click?
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.
Done.
README.md
Outdated
* [PASS Core](https://github.com/eclipse-pass/pass-core) | ||
* [PASS UI](https://github.com/eclipse-pass/pass-ui) | ||
* [PASS Support](https://github.com/eclipse-pass/pass-support) | ||
* [PASS Acceptance Testing](https://github.com/eclipse-pass) |
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.
Wrong URL
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.
Done.
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.
Looks good.
This PR will cleanup old documentation within the main repository. It updates the top level README with a brief summary and adds URLS pointing to the new documentation site. In addition the CONTRIBUTING.md will be updated to have a brief summary and add a URL pointing to the user docs contributing page. Once merged this will close #1035.
Summary of changes: