Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Support
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in / Register
Toggle navigation
G
gitlab-ce
Project overview
Project overview
Details
Activity
Releases
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Issues
0
Issues
0
List
Boards
Labels
Milestones
Merge Requests
1
Merge Requests
1
Analytics
Analytics
Repository
Value Stream
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Create a new issue
Commits
Issue Boards
Open sidebar
nexedi
gitlab-ce
Commits
d10ba344
Commit
d10ba344
authored
Sep 24, 2020
by
Erick Bajao
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
Remove old coverage feature flags
Removes ci_daily_code_coverage and ci_download_daily_code_coverage
parent
0709379d
Changes
5
Hide whitespace changes
Inline
Side-by-side
Showing
5 changed files
with
0 additions
and
22 deletions
+0
-22
app/controllers/projects/ci/daily_build_group_report_results_controller.rb
...rojects/ci/daily_build_group_report_results_controller.rb
+0
-5
app/controllers/projects/graphs_controller.rb
app/controllers/projects/graphs_controller.rb
+0
-1
app/services/ci/daily_build_group_report_result_service.rb
app/services/ci/daily_build_group_report_result_service.rb
+0
-2
config/feature_flags/development/ci_daily_code_coverage.yml
config/feature_flags/development/ci_daily_code_coverage.yml
+0
-7
config/feature_flags/development/ci_download_daily_code_coverage.yml
...ure_flags/development/ci_download_daily_code_coverage.yml
+0
-7
No files found.
app/controllers/projects/ci/daily_build_group_report_results_controller.rb
View file @
d10ba344
...
...
@@ -6,7 +6,6 @@ class Projects::Ci::DailyBuildGroupReportResultsController < Projects::Applicati
MAX_ITEMS
=
1000
REPORT_WINDOW
=
90
.
days
before_action
:validate_feature_flag!
before_action
:authorize_read_build_report_results!
before_action
:validate_param_type!
...
...
@@ -19,10 +18,6 @@ class Projects::Ci::DailyBuildGroupReportResultsController < Projects::Applicati
private
def
validate_feature_flag!
render_404
unless
Feature
.
enabled?
(
:ci_download_daily_code_coverage
,
project
,
default_enabled:
true
)
end
def
validate_param_type!
respond_422
unless
allowed_param_types
.
include?
(
param_type
)
end
...
...
app/controllers/projects/graphs_controller.rb
View file @
d10ba344
...
...
@@ -57,7 +57,6 @@ class Projects::GraphsController < Projects::ApplicationController
end
def
get_daily_coverage_options
return
unless
Feature
.
enabled?
(
:ci_download_daily_code_coverage
,
@project
,
default_enabled:
true
)
return
unless
can?
(
current_user
,
:read_build_report_results
,
project
)
date_today
=
Date
.
current
...
...
app/services/ci/daily_build_group_report_result_service.rb
View file @
d10ba344
...
...
@@ -3,8 +3,6 @@
module
Ci
class
DailyBuildGroupReportResultService
def
execute
(
pipeline
)
return
unless
Feature
.
enabled?
(
:ci_daily_code_coverage
,
pipeline
.
project
,
default_enabled:
true
)
DailyBuildGroupReportResult
.
upsert_reports
(
coverage_reports
(
pipeline
))
end
...
...
config/feature_flags/development/ci_daily_code_coverage.yml
deleted
100644 → 0
View file @
0709379d
---
name
:
ci_daily_code_coverage
introduced_by_url
:
rollout_issue_url
:
group
:
type
:
development
default_enabled
:
true
config/feature_flags/development/ci_download_daily_code_coverage.yml
deleted
100644 → 0
View file @
0709379d
---
name
:
ci_download_daily_code_coverage
introduced_by_url
:
rollout_issue_url
:
group
:
type
:
development
default_enabled
:
true
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment