| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
| |
'master'"
This reverts merge request !29733
|
| |\
| |
| |
| |
| |
| |
| | |
Use NotesFinder to fetch notes in module IssuableActions
Closes #52123
See merge request gitlab-org/gitlab-ce!29733
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Add spec for concern IssuableActions
Add shared samples for discussions endpoint
Add schema validations for discussions
Fix rubocop style issue
Make target assignable
Use new possibility to provide target
|
| |\ \
| | |
| | |
| | |
| | | |
Redirect project.wiki.git to project wiki home
See merge request gitlab-org/gitlab-ce!31085
|
| | | |
| | |
| | |
| | | |
https://gitlab.com/gitlab-org/gitlab-ce/issues/19186
|
| | | |
| | |
| | |
| | | |
Otherwise capybara hits ambiguous match.
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
Add frozen_string_literal to spec/lib (part 1)
See merge request gitlab-org/gitlab-ce!31130
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Note that Performance/UnfreezeString recommends unary plus over
"".dup, but unary plus has lower precedence so we have to use
parenthesis
|
| | |/ /
| | |
| | |
| | |
| | | |
Using the sed script from
https://gitlab.com/gitlab-org/gitlab-ce/issues/59758
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Make pipeline failure Slack notifications prettier and more informative
Closes #32495
See merge request gitlab-org/gitlab-ce!27683
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This commit adds some formatting to the Slack notifications for pipeline
statuses, as well as adds information about the stage and jobs
that failed in the case of pipeline failure.
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
Extract resource_label_events API shared examples
See merge request gitlab-org/gitlab-ce!31156
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
We don't need a repository, and we only need a private user in one
example, so we can scope it locally there.
|
| | | | | |
| | | | |
| | | | |
| | | | | |
This will allow us to reuse them in an EE-only spec.
|
| |\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Add frozen_string_literal to spec/factories
See merge request gitlab-org/gitlab-ce!31129
|
| | | |_|/ /
| |/| | |
| | | | |
| | | | |
| | | | | |
Using the sed script from
https://gitlab.com/gitlab-org/gitlab-ce/issues/59758
|
| | | | | | |
|
| |\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Remove code related to object hierarchy and MySQL
Closes #65056 and #65055
See merge request gitlab-org/gitlab-ce!31095
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
These are not required because MySQL is not
supported anymore
|
| |\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Find build by sha from ref
Closes #64534 and #45697
See merge request gitlab-org/gitlab-ce!30843
|
| | | | | | | | |
|
| | | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
* Reword Project#latest_successful_build_for to
Project#latest_successful_build_for_ref
* Reword Ci::Pipeline#latest_successful_for to
Ci::Pipeline#latest_successful_build_for_ref
|
| | | | | | | | |
|
| | | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Adds ability to find builds by sha when only specifying a ref.
|
| |\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
'master'
Remove fallback to deployment_platform_cluster in `DeploymentMetrics`
See merge request gitlab-org/gitlab-ce!31126
|
| | | |_|_|_|/ /
| |/| | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Deployment_platform is relatively expensive and calling this after the
fact means that this may not be the cluster that was deployed to.
Correspondingly reduce the leeway given in the related N+1 spec
|
| | | | | | | | |
|
| |\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Fix error rendering submodules in MR diffs when there is no .gitmodules
See merge request gitlab-org/gitlab-ce!31162
|
| | | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Without this change, we get a NoMethodError on nil
|
| | | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
The GitLab UI update changed the spinner selector from `spinner` to
`gl-spinner`, so we need to update it, so that our tests won't fail.
|
| | | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
It will be hooked up to the markdown editor later
|
| | |_|_|_|_|_|/
|/| | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
By not triggering the callback:
- ActiveSession lookup keys are not cleaned
- Devise also misses its hook related to session cleanup
|
| |\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Respect alt email when sending group access notifications
See merge request gitlab-org/gitlab-ce!31089
|
| | | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
When sending access granted/rejected emails we should also respect
custom emails set for groups/sub-groups
|
| | |_|_|_|_|/ /
|/| | | | | | |
|
| |\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
'master'
Resolve "Incorrect message interpolation on project listing"
Closes #65088
See merge request gitlab-org/gitlab-ce!31121
|
| | | |_|/ / / /
| |/| | | | |
| | | | | | |
| | | | | | | |
Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/65088
|
| |\ \ \ \ \ \ \
| |_|_|/ / / /
|/| | | | | |
| | | | | | |
| | | | | | | |
Logging sidekiq worker class name in SidekiqMemoryKiller
See merge request gitlab-org/gitlab-ce!30996
|
| | | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Currently, SidekiqMemoryKiller does not feed worker class name in the
json structured logging. This commit extends the json parameter.
|
| | | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
It needs to default to an empty array logically.
|
| | | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Added the ability to update the namespace label text for gitlab-managed
|
| |\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Add frozen_string_literal to spec/support
See merge request gitlab-org/gitlab-ce!31132
|
| | | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
This was shown in specs but surely this will be happening in application
code as well if this method is passes a frozen string.
We were also trying to force_encode a OmniAuth::AuthHash which had the
very confusing behaviour of returning nil when it was sent a method that
it did not define. Fix that by only force_encoding a String.
|
| | | | | | | | | |
|
| | | |/ / / / /
| |/| | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Using the sed script from
https://gitlab.com/gitlab-org/gitlab-ce/issues/59758
|
| |\ \ \ \ \ \ \
| |_|_|_|_|_|/
|/| | | | | |
| | | | | | |
| | | | | | | |
Support title and desc on merge w/ push option
See merge request gitlab-org/gitlab-ce!31068
|
| | | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
MergeRequests::PushOptionsHandlerService has been updated to allow
creating and updating merge requests with the `title` and
`description` set using git push options.
To create a new merge request and set its title and description:
git push -u origin -o merge_request.create \
-o merge_request.title="My title" \
-o merge_request.description="My description"
To update an existing merge request and set its title and
description:
git push -u origin -o merge_request.title="My title" \
-o merge_request.description="My description"
Issue https://gitlab.com/gitlab-org/gitlab-ce/issues/64320
|
| |\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Resolve "table_exists? not compatible with Rails 5.1"
Closes #54478
See merge request gitlab-org/gitlab-ce!30832
|
| | | |/ / / / /
| |/| | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Changes all calls to data_source_exists? to table_exists?
since that is the intent of these calls
|
| |\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Support Docker OCI images
Closes gitlab-ee#12877 and #58685
See merge request gitlab-org/gitlab-ce!31127
|