Commit ea498493 authored by David O'Regan's avatar David O'Regan

Merge branch 'pb-remove-jobs-page-ci-lint-button' into 'master'

Remove CI lint button from Jobs page nav

See merge request gitlab-org/gitlab!56854
parents 76c5f279 e2d6614d
...@@ -5,11 +5,5 @@ ...@@ -5,11 +5,5 @@
- build_path_proc = ->(scope) { project_jobs_path(@project, scope: scope) } - build_path_proc = ->(scope) { project_jobs_path(@project, scope: scope) }
= render "shared/builds/tabs", build_path_proc: build_path_proc, all_builds: @all_builds, scope: @scope = render "shared/builds/tabs", build_path_proc: build_path_proc, all_builds: @all_builds, scope: @scope
.nav-controls
- if can?(current_user, :update_build, @project)
= link_to project_ci_lint_path(@project), class: 'btn gl-button btn-default' do
%span
= _('CI Lint')
.content-list.builds-content-list .content-list.builds-content-list
= render "table", builds: @builds, project: @project = render "table", builds: @builds, project: @project
---
title: Remove CI lint button from Jobs page nav
merge_request: 56854
author:
type: removed
...@@ -24,14 +24,6 @@ RSpec.describe 'User browses jobs' do ...@@ -24,14 +24,6 @@ RSpec.describe 'User browses jobs' do
end end
end end
it 'shows the "CI Lint" button' do
page.within('.nav-controls') do
ci_lint_tool_link = page.find_link('CI Lint')
expect(ci_lint_tool_link[:href]).to end_with(project_ci_lint_path(project))
end
end
context 'with a failed job' do context 'with a failed job' do
let!(:build) { create(:ci_build, :coverage, :failed, pipeline: pipeline) } let!(:build) { create(:ci_build, :coverage, :failed, pipeline: pipeline) }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment