Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
李少辉-开发者
gitlab-foss
提交
ad8cd349
G
gitlab-foss
项目概览
李少辉-开发者
/
gitlab-foss
通知
15
Star
0
Fork
0
代码
文件
提交
分支
Tags
贡献者
分支图
Diff
Issue
0
列表
看板
标记
里程碑
合并请求
0
Wiki
0
Wiki
分析
仓库
DevOps
项目成员
Pages
G
gitlab-foss
项目概览
项目概览
详情
发布
仓库
仓库
文件
提交
分支
标签
贡献者
分支图
比较
Issue
0
Issue
0
列表
看板
标记
里程碑
合并请求
0
合并请求
0
Pages
分析
分析
仓库分析
DevOps
Wiki
0
Wiki
成员
成员
收起侧边栏
关闭侧边栏
动态
分支图
创建新Issue
提交
Issue看板
前往新版Gitcode,体验更适合开发者的 AI 搜索 >>
未验证
提交
ad8cd349
编写于
3月 01, 2018
作者:
R
Rémy Coutable
浏览文件
操作
浏览文件
下载
电子邮件补丁
差异文件
Document the ee-files-location-check CI job
Signed-off-by:
N
Rémy Coutable
<
remy@rymai.me
>
上级
bac9bb18
变更
1
隐藏空白更改
内联
并排
Showing
1 changed file
with
39 addition
and
0 deletion
+39
-0
doc/development/ee_features.md
doc/development/ee_features.md
+39
-0
未找到文件。
doc/development/ee_features.md
浏览文件 @
ad8cd349
...
...
@@ -33,6 +33,40 @@ rest of the code should be as close to the CE files as possible.
[
single code base
]:
https://gitlab.com/gitlab-org/gitlab-ee/issues/2952#note_41016454
### Detection of EE-only files
For each commit (except on
`master`
), the
`ee-files-location-check`
CI job tries
to detect if there are any new files that are EE-only. If any file is detected,
the job fails with an explanation of why and what to do to make it pass.
Basically, the fix is simple:
`git mv <file> ee/<file>`
.
#### How to name your branches?
For any EE branch, the job will try to detect its CE counterpart by removing any
`ee-`
prefix or
`-ee`
suffix from the EE branch name, and matching the last
branch that contains it.
For instance, from the EE branch
`new-shiny-feature-ee`
(or
`ee-new-shiny-feature`
), the job would find the corresponding CE branches:
-
`new-shiny-feature`
-
`ce-new-shiny-feature`
-
`new-shiny-feature-ce`
-
`my-super-new-shiny-feature-in-ce`
#### Whitelist some EE-only files that cannot be moved to `ee/`
The
`ee-files-location-check`
CI job provides a whitelist of files or folders
that cannot or should not be moved to
`ee/`
. Feel free to open an issue to
discuss adding a new file/folder to this whitelist.
For instance, it was decided that moving EE-only files from
`qa/`
to
`ee/qa/`
would make it difficult to build the
`gitLab-{ce,ee}-qa`
Docker images and it
was [not worth the complexity].
[
not worth the complexity
]:
https://gitlab.com/gitlab-org/gitlab-ee/issues/4997#note_59764702
### EE-only features
If the feature being developed is not present in any form in CE, we don't
...
...
@@ -52,6 +86,11 @@ is applied not only to models. Here's a list of other examples:
-
`ee/app/validators/foo_attr_validator.rb`
-
`ee/app/workers/foo_worker.rb`
This works because for every path that are present in CE's eager-load/auto-load
paths, we add the same
`ee/`
-prepended path in [
`config/application.rb`
].
[
`config/application.rb`
]:
https://gitlab.com/gitlab-org/gitlab-ee/blob/d278b76d6600a0e27d8019a0be27971ba23ab640/config/application.rb#L41-51
### EE features based on CE features
For features that build on existing CE features, write a module in the
...
...
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录