docs: remove redundant info & fix typo in pr template (#1997)

* fix pr template typo

* add contribution type link & remove reduntant info
status-code-override
Kagaya 3 years ago committed by chenzhihui
parent c329cffa3f
commit 0a8ab7a48a
  1. 26
      .github/pull_request_template.md

@ -3,22 +3,8 @@
1. If this is your first time contributing to Kratos, please read our contribution guide: https://go-kratos.dev/en/docs/community/contribution/ 1. If this is your first time contributing to Kratos, please read our contribution guide: https://go-kratos.dev/en/docs/community/contribution/
2. Ensure you have added or ran the appropriate tests and lint for your PR, please use `make lint` and `make test` before filing your PR, use `make clean` to tidy your go mod. 2. Ensure you have added or ran the appropriate tests and lint for your PR, please use `make lint` and `make test` before filing your PR, use `make clean` to tidy your go mod.
3. If the PR is unfinished, you may need mark it as a WIP(Work In Progress) PR or draft PR 3. If the PR is unfinished, you may need to mark it as a WIP(Work In Progress) PR or draft PR
4. Please use a conventional commits format title: `<type>[optional scope]: <description>` 4. Please use a semantic commits format title, such as `<type>[optional scope]: <description>`, see: https://go-kratos.dev/docs/community/contribution#type
Some suggestion on <type>:
fix: A bug fix
feat: A new feature
test: Adding missing tests or correcting existing tests
refactor: A code change that neither fixes a bug nor adds a feature
break: Changes has break change
docs: Documentation only changes
deps: Changes external dependencies
style: Changes that do not affect the meaning of the code (white-space, formatting, etc)
chore Daily work, examples, etc.
ci: Changes to our CI configuration files and scripts
--> -->
#### Description (what this PR does / why we need it): #### Description (what this PR does / why we need it):
@ -30,14 +16,14 @@
#### Which issue(s) this PR fixes (resolves / be part of): #### Which issue(s) this PR fixes (resolves / be part of):
<!-- <!--
* Automatically closes linked issue when PR is merged. * Automatically closes linked issue when PR is merged.
* If you PR is not fully resolved issue, please use `part of #<issue number>` instead. * If your PR is not fully resolved the issue, please use `part of #<issue number>` instead.
Usage: `fixes/resolves #<issue number>`, or `fixes/resolves (paste link of issue)`. Usage: `fixes/resolves #<issue number>`, or `fixes/resolves (paste link of issue)`.
--> -->
fixes #
#### Other special notes for reviewer:
#### Other special notes for the reviewers:
<!-- <!--
* Somethings that need extra attention for reviewer * Somethings that need extra attention for the reviewers
* Some additional notes, TODO list, etc. * Some additional notes, TODO list, etc.
--> -->
Loading…
Cancel
Save