Skip to content
代码片段 群组 项目
提交 16b65cce 编辑于 作者: Mark Chao's avatar Mark Chao
浏览文件

Merge branch 'furkanayhan-master-patch-39633' into 'master'

Add more options for feature flag changelogs in docs

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/98095



Merged-by: default avatarMark Chao <mchao@gitlab.com>
Approved-by: default avatarMark Chao <mchao@gitlab.com>
Co-authored-by: default avatarFurkan Ayhan <furkanayhn@gmail.com>
No related branches found
No related tags found
无相关合并请求
......@@ -535,16 +535,18 @@ Feature.remove(:feature_flag_name)
```mermaid
graph LR
A[flag: default off] -->|'added' / 'changed'| B(flag: default on)
A[flag: default off] -->|'added' / 'changed' / 'fixed' / '...'| B(flag: default on)
B -->|'other'| C(remove flag, keep new code)
B -->|'removed' / 'changed'| D(remove flag, keep old code)
A -->|'added' / 'changed'| C
A -->|'added' / 'changed' / 'fixed' / '...'| C
A -->|no changelog| D
```
- Any change behind a feature flag that is **enabled** by default **should** have a changelog entry.
- The changelog for a feature flag should describe the feature and not the
flag, unless a default on feature flag is removed keeping the new code (`other` in the flowchart above).
- A feature flag can also be used for rolling out a bug fix or a maintenance work. In this scenario, the changelog
must be related to it, for example; `fixed` or `other`.
## Feature flags in tests
......
0% 加载中 .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册