-
Notifications
You must be signed in to change notification settings - Fork 50
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
Add contributing guide #67
Comments
Oh, I just realized that @taylorsilva just had #65 in 🤔 |
This is a great idea if there are other things to capture. Running that script for the job templates is only one thing, but we also have the entire What other items could go into a |
yeah, perhaps documenting the
(see https://github.com/concourse/concourse/blob/master/CONTRIBUTING.md#rebuilding-to-test-your-changes) |
oh, something else: giving some guidance on adding things like |
Feature Request
What challenge are you facing?
There are certain steps that we need to go through before submitting a PR to this repository, e.g., ensuring that the
erb
templates were properly generated.Currently, that's "tribal knowledge" spread across the team that is not openly communicated (and sometimes, forgotten by us 😅 )
A Modest Proposal
Have a
CONTRIBUTING.md
file, just like we do inconcourse/concourse
(seecontributing.md
).The text was updated successfully, but these errors were encountered: