mirror of
https://github.com/karl0ss/homepage.git
synced 2025-05-02 21:43:39 +01:00
Update guidelines
This commit is contained in:
parent
4c45c6453f
commit
b2a914eb2a
4
.github/PULL_REQUEST_TEMPLATE.md
vendored
4
.github/PULL_REQUEST_TEMPLATE.md
vendored
@ -25,7 +25,7 @@ What type of change does your PR introduce to Homepage?
|
|||||||
|
|
||||||
## Checklist:
|
## Checklist:
|
||||||
|
|
||||||
- [ ] If adding a service widget or a change that requires it, I have added corresponding documentation changes.
|
- [ ] If applicable, I have added corresponding documentation changes.
|
||||||
- [ ] If adding a new widget I have reviewed the [guidelines](https://gethomepage.dev/latest/more/development/#service-widget-guidelines).
|
- [ ] If applicable, I have reviewed the [feature](https://gethomepage.dev/latest/more/development/#new-feature-guidelines) and / or [service widget guidelines](https://gethomepage.dev/latest/more/development/#service-widget-guidelines).
|
||||||
- [ ] I have checked that all code style checks pass using [pre-commit hooks](https://gethomepage.dev/latest/more/development/#code-formatting-with-pre-commit-hooks) and [linting checks](https://gethomepage.dev/latest/more/development/#code-linting).
|
- [ ] I have checked that all code style checks pass using [pre-commit hooks](https://gethomepage.dev/latest/more/development/#code-formatting-with-pre-commit-hooks) and [linting checks](https://gethomepage.dev/latest/more/development/#code-linting).
|
||||||
- [ ] If applicable, I have tested my code for new features & regressions on both mobile & desktop devices, using the latest version of major browsers.
|
- [ ] If applicable, I have tested my code for new features & regressions on both mobile & desktop devices, using the latest version of major browsers.
|
||||||
|
@ -39,11 +39,16 @@ Once installed, hooks will run when you commit. If the formatting isn't quite ri
|
|||||||
|
|
||||||
See the [pre-commit documentation](https://pre-commit.com/#install) to get started.
|
See the [pre-commit documentation](https://pre-commit.com/#install) to get started.
|
||||||
|
|
||||||
|
## New Feature Guidelines
|
||||||
|
|
||||||
|
- New features should be linked to an existing feature request with at least 5 'up-votes'. The purpose of this requirement is to avoid the addition (and maintenance) of features that might only benefit a small number of users.
|
||||||
|
- If you have ideas for a larger feature, please open a discussion first.
|
||||||
|
|
||||||
## Service Widget Guidelines
|
## Service Widget Guidelines
|
||||||
|
|
||||||
To ensure cohesiveness of various widgets, the following should be used as a guide for developing new widgets:
|
To ensure cohesiveness of various widgets, the following should be used as a guide for developing new widgets:
|
||||||
|
|
||||||
- Please only submit widgets that have been requested and have at least 5 'up-votes'
|
- Please only submit widgets that have been requested and have at least 5 'up-votes'. The purpose of this requirement is to avoid the addition (and maintenance) of service widgets that might only benefit a small number of users.
|
||||||
- Widgets should be only one row of blocks
|
- Widgets should be only one row of blocks
|
||||||
- Widgets should be no more than 4 blocks wide
|
- Widgets should be no more than 4 blocks wide
|
||||||
- Minimize the number of API calls
|
- Minimize the number of API calls
|
||||||
|
Loading…
x
Reference in New Issue
Block a user