Update files when snippet is updated
What does this MR do?
When a snippet is updated we have to update the repository content as well. In this MR, we add the necessary actions to update the file path or content in the repository.
Refs #39265 (closed) and #199221 (closed)
Screenshots
Does this MR meet the acceptance criteria?
Conformity
Merge request reports
Activity
changed milestone to %12.8
added Deliverable backend devopscreate groupeditor [DEPRECATED] typefeature + 1 deleted label
added 2 commits
added 74 commits
-
100d58ee...3c113463 - 73 commits from branch
fj-39265-create-snippet-repository-content - 1b27eba1 - backup
-
100d58ee...3c113463 - 73 commits from branch
removed Deliverable label
changed milestone to %12.9
mentioned in merge request !23953 (merged)
added 2696 commits
-
22aa0676...ef6f113d - 2692 commits from branch
fj-39265-create-snippet-repository-content - e913d1c1 - Create snippet repository models
- e4854f98 - Commit file when snippet is created
- ff6325c9 - Commit file when snippet is update
- 97a92899 - Add spec
Toggle commit list-
22aa0676...ef6f113d - 2692 commits from branch
added 1 commit
- 44b058fe - Update snippet repository when snippet updated
added 715 commits
-
44b058fe...9338931a - 714 commits from branch
master - e1131d14 - Update snippet repository when snippet updated
-
44b058fe...9338931a - 714 commits from branch
added 192 commits
-
e1131d14...885e4d44 - 191 commits from branch
master - 0abc41f7 - Update snippet repository when snippet updated
-
e1131d14...885e4d44 - 191 commits from branch
added database databasereview pending labels
added 415 commits
-
63487051...1d9484d1 - 411 commits from branch
master - 0d00d9d4 - Update snippet repository when snippet updated
- ff97256f - Adjust logic and fix specs
- 16db3475 - Creating repository if not exists
- 3d36c7be - Small changes and added specs
Toggle commit list-
63487051...1d9484d1 - 411 commits from branch
mentioned in issue #199221 (closed)
added 133 commits
Toggle commit listadded 1 commit
- d81508fb - Update snippet repository when snippet updated
marked the checklist item Changelog entry as completed
removed database databasereview pending labels
-
@mbobin do you mind to review?
assigned to @mbobin and unassigned @fjsanpedro
added 203 commits
-
d81508fb...5b12631c - 202 commits from branch
master - a689364b - Update snippet repository when snippet updated
-
d81508fb...5b12631c - 202 commits from branch
-
Sorry @mbobin I have to reassign this to somebody in my group to follow the rules of a pilot we're running.
@vhawoldar do you mind to review?
assigned to @vhawoldar and unassigned @mbobin
added 103 commits
-
63971926...36029973 - 101 commits from branch
master - 5deec8e5 - Update snippet repository when snippet updated
- 03e0e2ff - Fixing spec
-
63971926...36029973 - 101 commits from branch
@fjsanpedro this is looking great, I've added a few questions but I don't think any of them are really blocking this
assigned to @fjsanpedro and unassigned @vhawoldar
assigned to @vhawoldar and unassigned @fjsanpedro
added 1 commit
- 62da4e05 - Update snippet repository when snippet updated
@fjsanpedro LGTM!
assigned to @fjsanpedro and unassigned @vhawoldar
-
Thanks @vhawoldar! @smcgivern do you mind to review?
assigned to @smcgivern and unassigned @fjsanpedro
added 128 commits
-
62da4e05...8da519a8 - 127 commits from branch
master - 5c5de8fa - Update snippet repository when snippet updated
-
62da4e05...8da519a8 - 127 commits from branch
-
Thanks @fjsanpedro @vhawoldar, looks good! Just a couple of questions.
assigned to @fjsanpedro and unassigned @smcgivern
added 1 commit
- 94cbb5db - Update snippet repository when snippet updated
Reviewer roulette
Changes that require review have been detected! A merge request is normally reviewed by both a reviewer and a maintainer in its primary category (e.g. frontend or backend), and by a maintainer in all other categories.
To spread load more evenly across eligible reviewers, Danger has randomly picked a candidate for each review slot. Feel free to override this selection if you think someone else would be better-suited, or the chosen person is unavailable.
To read more on how to use the reviewer roulette, please take a look at the Engineering workflow and code review guidelines.
Once you've decided who will review this merge request, mention them as you normally would! Danger does not (yet?) automatically notify them for you.
Category Reviewer Maintainer backend Erick Bajao ( @iamricecake)Sean McGivern ( @smcgivern)Generated by
-
@smcgivern I addresses/answered your comments. Another look?
assigned to @smcgivern and unassigned @fjsanpedro
-
Thanks @fjsanpedro!
-
Thanks @smcgivern. The Merge widget is raising the error
Merge failed: Something went wrong during merge pre-receive hook. API is not accessible. Please try again.
mentioned in commit 84836fef
mentioned in issue #18079
added workflowverification label
mentioned in issue gitlab-com/gl-infra/scalability#190
added workflowstaging label and removed workflowverification label
added workflowcanary label and removed workflowstaging label
added workflowproduction label and removed workflowcanary label
added Category:Source Code Management snippets labels and removed 1 deleted label
