Skip to content

Commit 07959a9

Browse files
author
GitLab Bot
committed
Add latest changes from gitlab-org/gitlab@master
1 parent 92fa4c9 commit 07959a9

File tree

61 files changed

+705
-1115
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

61 files changed

+705
-1115
lines changed

.rubocop_todo/capybara/visibility_matcher.yml

-1
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,6 @@ Capybara/VisibilityMatcher:
3434
- 'spec/features/merge_request/user_views_diffs_commit_spec.rb'
3535
- 'spec/features/merge_request/user_views_diffs_spec.rb'
3636
- 'spec/features/projects/blobs/blob_show_spec.rb'
37-
- 'spec/features/projects/blobs/user_follows_pipeline_suggest_nudge_spec.rb'
3837
- 'spec/features/projects/ci/lint_spec.rb'
3938
- 'spec/features/projects/commit/comments/user_adds_comment_spec.rb'
4039
- 'spec/features/projects/commits/multi_view_diff_spec.rb'

.rubocop_todo/layout/line_length.yml

-1
Original file line numberDiff line numberDiff line change
@@ -3117,7 +3117,6 @@ Layout/LineLength:
31173117
- 'spec/features/projects/blobs/blob_show_spec.rb'
31183118
- 'spec/features/projects/blobs/edit_spec.rb'
31193119
- 'spec/features/projects/blobs/shortcuts_blob_spec.rb'
3120-
- 'spec/features/projects/blobs/user_follows_pipeline_suggest_nudge_spec.rb'
31213120
- 'spec/features/projects/blobs/user_views_pipeline_editor_button_spec.rb'
31223121
- 'spec/features/projects/ci/editor_spec.rb'
31233122
- 'spec/features/projects/commit/cherry_pick_spec.rb'

app/assets/javascripts/blob/filepath_form/components/filepath_form.vue

-6
Original file line numberDiff line numberDiff line change
@@ -21,11 +21,6 @@ export default {
2121
type: Object,
2222
required: true,
2323
},
24-
suggestCiYmlData: {
25-
type: Object,
26-
required: false,
27-
default: undefined,
28-
},
2924
},
3025
data() {
3126
return {
@@ -56,7 +51,6 @@ export default {
5651
:filename="filename"
5752
:templates="templates"
5853
:initial-template="initialTemplate"
59-
:suggest-ci-yml-data="suggestCiYmlData"
6054
@selected="onTemplateSelected"
6155
/>
6256
</div>

app/assets/javascripts/blob/filepath_form/components/template_selector.vue

-18
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
<script>
22
import { GlCollapsibleListbox } from '@gitlab/ui';
3-
import SuggestGitlabCiYml from '~/blob/suggest_gitlab_ci_yml/components/popover.vue';
43
import { __ } from '~/locale';
54
import { DEFAULT_CI_CONFIG_PATH, CI_CONFIG_PATH_EXTENSION } from '~/lib/utils/constants';
65
@@ -34,7 +33,6 @@ const templateSelectors = [
3433
export default {
3534
name: 'TemplateSelector',
3635
components: {
37-
SuggestGitlabCiYml,
3836
GlCollapsibleListbox,
3937
},
4038
props: {
@@ -51,11 +49,6 @@ export default {
5149
required: false,
5250
default: undefined,
5351
},
54-
suggestCiYmlData: {
55-
type: Object,
56-
required: false,
57-
default: undefined,
58-
},
5952
},
6053
data() {
6154
return {
@@ -97,9 +90,6 @@ export default {
9790
showDropdown() {
9891
return this.activeType && this.templateItems.length > 0;
9992
},
100-
showPopover() {
101-
return this.activeType?.key === 'gitlab_ci_ymls' && this.suggestCiYmlData;
102-
},
10393
},
10494
beforeMount() {
10595
if (this.activeType) this.applyTemplate(this.initialTemplate);
@@ -135,14 +125,6 @@ export default {
135125
</script>
136126
<template>
137127
<div v-if="showDropdown">
138-
<suggest-gitlab-ci-yml
139-
v-if="showPopover"
140-
target="template-selector"
141-
:track-label="suggestCiYmlData.trackLabel"
142-
:dismiss-key="suggestCiYmlData.dismissKey"
143-
:merge-request-path="suggestCiYmlData.mergeRequestPath"
144-
:human-access="suggestCiYmlData.humanAccess"
145-
/>
146128
<gl-collapsible-listbox
147129
id="template-selector"
148130
searchable

app/assets/javascripts/blob/filepath_form/index.js

-11
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,6 @@
11
import Vue from 'vue';
22
import FilepathForm from './components/filepath_form.vue';
33

4-
const getPopoverData = (el) => ({
5-
trackLabel: el.dataset.trackLabel,
6-
dismissKey: el.dataset.dismissKey,
7-
mergeRequestPath: el.dataset.mergeRequestPath,
8-
humanAccess: el.dataset.humanAccess,
9-
});
10-
114
const getInputOptions = (el) => {
125
const { testid, qa_selector: qaSelector, ...options } = JSON.parse(el.dataset.inputOptions);
136
return {
@@ -19,15 +12,11 @@ const getInputOptions = (el) => {
1912
export default ({ onTemplateSelected }) => {
2013
const el = document.getElementById('js-template-selectors-menu');
2114

22-
const suggestCiYmlEl = document.querySelector('.js-suggest-gitlab-ci-yml');
23-
const suggestCiYmlData = suggestCiYmlEl ? getPopoverData(suggestCiYmlEl) : undefined;
24-
2515
return new Vue({
2616
el,
2717
render(h) {
2818
return h(FilepathForm, {
2919
props: {
30-
suggestCiYmlData,
3120
inputOptions: getInputOptions(el),
3221
templates: JSON.parse(el.dataset.templates),
3322
initialTemplate: el.dataset.selected,

app/assets/javascripts/blob/filepath_form_mediator.js

-6
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
import $ from 'jquery';
22

33
import Api from '~/api';
4-
import initPopover from '~/blob/suggest_gitlab_ci_yml';
54
import { createAlert } from '~/alert';
65
import { __, sprintf } from '~/locale';
76
import toast from '~/vue_shared/plugins/global_toast';
@@ -33,7 +32,6 @@ export default class FilepathFormMediator {
3332

3433
selectTemplateFile(template, type, clearSelectedTemplate, stopLoading) {
3534
const self = this;
36-
const suggestCommitChanges = document.querySelector('.js-suggest-gitlab-ci-yml-commit-changes');
3735

3836
this.fetchFileTemplate(type.type, template.key, template)
3937
.then((file) => {
@@ -50,10 +48,6 @@ export default class FilepathFormMediator {
5048
},
5149
},
5250
});
53-
54-
if (suggestCommitChanges) {
55-
initPopover(suggestCommitChanges);
56-
}
5751
})
5852
.catch((err) =>
5953
createAlert({

app/assets/javascripts/blob/pipeline_tour_success_modal.vue

-143
This file was deleted.

0 commit comments

Comments
 (0)