forked from FoundKeyGang/FoundKey
Update CONTRIBUTING.md
This commit is contained in:
parent
ec23e6270b
commit
b590f352e9
1 changed files with 15 additions and 0 deletions
|
@ -42,6 +42,21 @@ Thank you for your PR! Before creating a PR, please check the following:
|
||||||
|
|
||||||
Thanks for your cooperation 🤗
|
Thanks for your cooperation 🤗
|
||||||
|
|
||||||
|
## Reviewers guide
|
||||||
|
### Review perspective
|
||||||
|
- Scope
|
||||||
|
- Are the goals of the PR clear?
|
||||||
|
- Is the granularity of the PR appropriate?
|
||||||
|
- Security
|
||||||
|
- Does merging this PR create a vulnerability?
|
||||||
|
- Performance
|
||||||
|
- Will merging this PR cause unexpected performance degradation?
|
||||||
|
- Is there a more efficient way?
|
||||||
|
- Testing
|
||||||
|
- Does the test ensure the expected behavior?
|
||||||
|
- Are there any omissions or gaps?
|
||||||
|
- Does it check for anomalies?
|
||||||
|
|
||||||
## Localization (l10n)
|
## Localization (l10n)
|
||||||
Misskey uses [Crowdin](https://crowdin.com/project/misskey) for localization management.
|
Misskey uses [Crowdin](https://crowdin.com/project/misskey) for localization management.
|
||||||
You can improve our translations with your Crowdin account.
|
You can improve our translations with your Crowdin account.
|
||||||
|
|
Loading…
Reference in a new issue