diff options
author | Winnie Hellmann <winnie@gitlab.com> | 2018-06-11 20:15:38 +0200 |
---|---|---|
committer | Winnie Hellmann <winnie@gitlab.com> | 2018-06-11 20:15:38 +0200 |
commit | 988d8a08e14d49287f53dd3bbbd014a883c16f5f (patch) | |
tree | 31ccda7243a4170bfdd29e6631ea74d39ef785ab | |
parent | 75797ac3d2b534a1deda48c8450027055a7c721b (diff) | |
download | gitlab-ce-winh-separate-refactorings-deliverables-docs.tar.gz |
Add rule to keep refactorings and deliverables separate to frontend guidewinh-separate-refactorings-deliverables-docs
-rw-r--r-- | doc/development/fe_guide/development_process.md | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/doc/development/fe_guide/development_process.md b/doc/development/fe_guide/development_process.md index d240dbe8b02..432747938ee 100644 --- a/doc/development/fe_guide/development_process.md +++ b/doc/development/fe_guide/development_process.md @@ -70,6 +70,24 @@ GitLab's architecture. 1. Don't take more than one week between starting work on a feature and sharing a Merge Request with a reviewer or a maintainer. +### Keep refactorings and deliverables separate + +In general, new features and bug fixes should not happen in the same merge request as refactorings. +This ensures we are not blocking a deliverable because the path of refactoring needs to be adjusted or has external dependencies. +If a refactoring makes sense in the context of a deliverable, make sure to start it before you work on the deliverable itself. +That way you can better decide early in the cycle if the refactoring is likely to be completed on time. + +#### Requirements for using a new class, component, or paradigm in a deliverable + +- There is at least one example in the code base already (by refactoring an existing feature). +- The new way of doing things is at least mentioned in the frontend guide. +- The change was announced in a frontend call. + +#### Exceptions from this rule + +- The refactoring is immediately necessary to proceed with the deliverable. +- The refactoring is necessary to have the deliverable ready before feature freeze. + ### Vue features 1. Follow the steps in [Vue.js Best Practices](vue.md) 1. Follow the style guide. |