-
Notifications
You must be signed in to change notification settings - Fork 496
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 an athens philosophy document #246
Conversation
Codecov Report
@@ Coverage Diff @@
## master #246 +/- ##
=======================================
Coverage 41.58% 41.58%
=======================================
Files 79 79
Lines 2032 2032
=======================================
Hits 845 845
Misses 1107 1107
Partials 80 80 Continue to review full report at Codecov.
|
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.
This is awesome 👍
README.md
Outdated
@@ -43,8 +43,11 @@ This project is early and there's plenty of interesting and challenging work to | |||
If you find a bug or want to fix a bug, we :heart: PRs and issues! If you see an issue | |||
in the [queue](https://github.com/gomods/athens/issues) that you'd like to work on, please just post a comment saying that you want to work on it. Something like "I want to work on this" is fine. | |||
|
|||
If you decide to contribute (we hope you do :smile:), the process is familiar and easy if you've used Github before. | |||
There are no long documents to read or complex setup. | |||
If you decide to contribute (we hope you do :smile:), the process is familiar and easy if you've used Github before. There are no long documents to read or complex setup. |
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.
can you attach brians video for first PR: https://www.youtube.com/watch?v=bgSDcTyysRc
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 LOVE IT!!!
PHILOSOPHY.md
Outdated
|
||
# Guiding Principles | ||
|
||
This is the TL;DR of the whole document! The Athens project has guiding principles: |
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.
maybe a small change here like s/project has guiding principles/project has {some, a few} guiding principles/
README.md
Outdated
There are no long documents to read or complex setup. | ||
If you decide to contribute (we hope you do :smile:), the process is familiar and easy if you've used Github before. There are no long documents to read or complex setup. | ||
|
||
Before you do start contributing or otherwise getting involved, we want to let you know that we follow a general [philosophy](./PHILOSOPHY.md) in how we work together, and we'd really appreciate you to get familiar with it before you start. |
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.
maybe s/appreciate you to get familiar/appreciate you getting familiar/ or similar
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.
Just two silly grammar-ish things if you agree. Great idea to add it though.
thanks @robjloranger!
@robjloranger just pushed the grammar fixes. @michalpristas I'm watching the Go OSS contribution video now 😄 |
@michalpristas added the OSS contribution video. I ❤️ it |
I'm attempting here to document generally how we like to work and set some standards so that newcomers know what to expect when they join our community.