| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
| |
It seems the deploy function causes the job to fail if it doesn't
succeed. That wasn't the intent as we want to curl the Review App after
the deploy finished (even if it failed) because sometimes the Review App
is just a bit long to be ready.
This change wraps the Review App deployment with "set +e"/"set -e" to
ensure that the job doesn't fail right away if the deploy fails.
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
|\
| |
| |
| |
| | |
Removes ifEE example for JS code.
See merge request gitlab-org/gitlab-ce!27408
|
| | |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Add help texts to K8 form fields
Closes #59275
See merge request gitlab-org/gitlab-ce!27274
|
|/ / |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
'master'
Fix logic in Api::Internal test
See merge request gitlab-org/gitlab-ce!27193
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The intention of this test is to ensure that the service class
MergeRequests::PushOptionsHandlerService does not run when the
:mr_push_options feature flag is disabled.
This test was passing, however was not testing what it was supposed to
be!
For one, setting Feature.disable(:feature) in the test does not disable
the feature, as rspec config in spec_helper stubs Feature to make all
features enabled:
https://gitlab.com/gitlab-org/gitlab-ce/commit/3ee48e422defaedd69946c607bd8d3672e510375
So the feature was still enabled in the test.
But this test wasn't failing because unfortunately I had put:
```
expect(MergeRequests::PushOptionsHandlerService).to receive(:new)
```
instead of not_to!
This meant that the `.new` method was being stubbed, so the service
class did not create a MergeRequest, which satisfied the second
expectation.
```
expect(MergeRequests::PushOptionsHandlerService).to receive(:new)
```
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
'master'
Allow extra args for helm in Auto-DevOps
See merge request gitlab-org/gitlab-ce!26838
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | | |
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
[CE backport] Fix no assignee text in sidebar_spec
See merge request gitlab-org/gitlab-ce!27444
|
|/ / / / |
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Set correct timestamps when creating past issues
Closes #58361
See merge request gitlab-org/gitlab-ce!27406
|
| | | | |
| | | | |
| | | | |
| | | | | |
Sets `system_note_timestamp` from `created_at`
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
`None` for assignees
Closes #60599
See merge request gitlab-org/gitlab-ce!27419
|
|/ / / / /
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Update assignees_spec.js
Update sidebar_spec.rb
Update issues_spec.rb
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Test using Git 2.21
See merge request gitlab-org/gitlab-ce!27418
|
| | | | | | |
|
| | | | | | |
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Given Rails doesn't use the Git binary anymore, this shouldn't be a
problem.
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Resolve "Karma failing with MrWidgetAlertMessage when helpPath is provided Maximum call stack size exceeded"
Closes #60605
See merge request gitlab-org/gitlab-ce!27429
|
|/ / / / / / |
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Docs: CI/CD intro - add new illustration + description
See merge request gitlab-org/gitlab-ce!27278
|
|/ / / / / / |
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Link to resource label events API
See merge request gitlab-org/gitlab-ce!27412
|
|/ / / / / / |
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Create frontend FAQ (documentation)
See merge request gitlab-org/gitlab-ce!26372
|
|/ / / / / / |
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Replaced "View file" button with doc-text icon
Closes #59034
See merge request gitlab-org/gitlab-ce!27365
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/59034
|
|\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
into 'master'
Update merge request widget to accommodate post-merge pipelines
See merge request gitlab-org/gitlab-ce!25983
|
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
This commit adds two new warning messages to the MR widget that handle
cases involving merge request pipelines.
|
|\ \ \ \ \ \ \ \
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Fix minor typo in LDAP docs
Closes #60413
See merge request gitlab-org/gitlab-ce!27415
|
| | | | | | | | | |
|
|\ \ \ \ \ \ \ \ \
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
Fix styling for `app/assets/stylesheets/pages/status.scss`
Closes #59929
See merge request gitlab-org/gitlab-ce!26875
|
| | |_|_|_|_|_|_|/
| |/| | | | | | | |
|
|\ \ \ \ \ \ \ \ \
| |_|_|_|_|/ / / /
|/| | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Hide overflow in scoped labels
Closes #48431
See merge request gitlab-org/gitlab-ce!27362
|
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Scoped labels overflew in all the places because of
lack of restriction on `max-width`. This commit introduces
the same limits on width we have inplace for regular labels
|
|\ \ \ \ \ \ \ \ \
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
Backport metrics report type
See merge request gitlab-org/gitlab-ce!26798
|
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
This backports the changes from
https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/10452
|
|\ \ \ \ \ \ \ \ \ \
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | | |
Update migration docs
See merge request gitlab-org/gitlab-ce!27261
|
|/ / / / / / / / / / |
|
|\ \ \ \ \ \ \ \ \ \
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | | |
Docs: Complete "Repository storage" directions
See merge request gitlab-org/gitlab-ce!27346
|
| | |/ / / / / / / /
| |/| | | | | | | | |
|
|\ \ \ \ \ \ \ \ \ \
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | | |
Resolve "Tooltip for "View file" shows raw HTML"
Closes #60533
See merge request gitlab-org/gitlab-ce!27398
|
| | |_|_|_|_|_|_|_|/
| |/| | | | | | | | |
|
|\ \ \ \ \ \ \ \ \ \
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | | |
Extract move quick action spec to shared example
Closes #59570
See merge request gitlab-org/gitlab-ce!26907
|