mirror of
https://github.com/tektoncd/catalog.git
synced 2024-11-26 06:23:37 +00:00
Remove "lint" checklist item 📝
To make sure the checklist is useful and folks don't ignore it, I think it's important to keep only the items we need to in there. If we can enforce something with CI, I suggest we do not include it in the checklist - if we want to write down guidance somewhere else I think that totally makes sense, but not in the checklist. From https://github.com/tektoncd/catalog/issues/387 it looks like linting should be applied automatically to each PR so I don't think we need to include this checklist item.
This commit is contained in:
parent
3c23c446a9
commit
afdb1215cd
1
.github/pull_request_template.md
vendored
1
.github/pull_request_template.md
vendored
@ -12,7 +12,6 @@ review them:
|
||||
|
||||
- [ ] Includes [docs](https://github.com/tektoncd/community/blob/master/standards.md#principles) (if user facing)
|
||||
- [ ] Commit messages follow [commit message best practices](https://github.com/tektoncd/community/blob/master/standards.md#commit-messages)
|
||||
- [ ] Yaml file complies with [yamllint](https://github.com/adrienverge/yamllint) rules.
|
||||
- [ ] Complies with [Catalog Orgainization TEP][TEP], see [example]. **Note** [An issue has been filed to automate this validation][validation]
|
||||
- [ ] File path follows `<kind>/<name>/<version>/name.yaml`
|
||||
- [ ] Has `README.md` at `<kind>/<name>/<version>/README.md`
|
||||
|
Loading…
Reference in New Issue
Block a user