Age | Commit message (Collapse) | Author |
|
Fix deprecation: Passing conditions to delete_all is deprecated
See merge request gitlab-org/gitlab-ce!23817
|
|
Fix Issue boards don't show milestone in issue details
See merge request gitlab-org/gitlab-ce!23813
|
|
* dev/master:
Update CHANGELOG.md for 11.3.13
Validate LFS hrefs before downloading them
|
|
[MASTER]: Validate LFS hrefs before downloading them
Closes #2754
See merge request gitlab/gitlabhq!2696
|
|
|
|
|
|
|
|
Resolve "Fix labels dropdown with multiple same names"
Closes #51994
See merge request gitlab-org/gitlab-ce!23265
|
|
Fix diff changes empty state
Closes #48635
See merge request gitlab-org/gitlab-ce!23767
|
|
|
|
Restore Object Pools when restoring an object pool
Closes gitaly#1355
See merge request gitlab-org/gitlab-ce!23682
|
|
The empty state now only gets shown when no files exist in the branch.
If the user is reviewing 2 versions with no files, we don't show the state.
Refactors the diff app spec to use Vue test utils.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/48635
|
|
|
|
Pool repositories are persisted in the database, and when the DB is
restored, the data need to be restored on disk. This is done by
resetting the state machine and rescheduling the object pool creation.
This is not an exact replica of the state like at the time of the
creation of the backup. However, the data is consistent again.
Dumping isn't required as internally GitLab uses git bundles which
bundle all refs and include all objects in the bundle that they require,
reduplicating as more repositories get backed up. This does require more
data to be stored.
Fixes https://gitlab.com/gitlab-org/gitaly/issues/1355
|
|
|
|
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/55248, https://gitlab.com/gitlab-org/gitlab-ce/issues/55245
|
|
Resolve various database deprecation warnings
See merge request gitlab-org/gitlab-ce!23772
|
|
Refactor Project#create_or_update_import_data
See merge request gitlab-org/gitlab-ce!23701
|
|
|
|
Extending from ActiveRecord::Migration is deprecated, but was still used
in a bunch of places.
|
|
Add basic implementation of CI/CD bridge job
See merge request gitlab-org/gitlab-ce!23730
|
|
|
|
'54650-send-an-email-to-project-owners-when-a-mirror-update-fails' into 'master'
Send a notification email on mirror update errors
Closes #54650
See merge request gitlab-org/gitlab-ce!23595
|
|
Fix issuables sort direction button parameters
Closes #55133
See merge request gitlab-org/gitlab-ce!23744
|
|
Fixed emoji awards not being added to notes
Closes #55141
See merge request gitlab-org/gitlab-ce!23720
|
|
|
|
Remove unnecessary div from MarkdownField to apply list styles correctly
Closes #55190
See merge request gitlab-org/gitlab-ce!23733
|
|
|
|
The email is sent to project maintainers containing the last mirror
update error. This will allow maintainers to set alarms and react
accordingly.
|
|
'55183-frozenerror-can-t-modify-frozen-string-in-app-mailers-notify-rb' into 'master'
Fix a potential frozen string error in app/mailers/notify.rb
Closes #55183
See merge request gitlab-org/gitlab-ce!23728
|
|
Migrate timeline_entry_item_spec.js to Jest
See merge request gitlab-org/gitlab-ce!23686
|
|
|
|
Fix labels parameter being deleted from filter
when clicking sort direction button on issues/merge requests search bar.
|
|
|
|
Generalise test compare service
See merge request gitlab-org/gitlab-ce!22833
|
|
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
|
|
|
|
|
In https://gitlab.com/gitlab-org/release/framework/issues/28 we found
that this method was changed a lot over the years: 43 times if our
calculations were correct. Looking at the method, it had quite a few
branches going on:
def create_or_update_import_data(data: nil, credentials: nil)
return if data.nil? && credentials.nil?
project_import_data = import_data || build_import_data
if data
project_import_data.data ||= {}
project_import_data.data = project_import_data.data.merge(data)
end
if credentials
project_import_data.credentials ||= {}
project_import_data.credentials =
project_import_data.credentials.merge(credentials)
end
project_import_data
end
If we turn the || and ||= operators into regular if statements, we can
see a bit more clearly that this method has quite a lot of branches in
it:
def create_or_update_import_data(data: nil, credentials: nil)
if data.nil? && credentials.nil?
return
else
project_import_data =
if import_data
import_data
else
build_import_data
end
if data
if project_import_data.data
# nothing
else
project_import_data.data = {}
end
project_import_data.data =
project_import_data.data.merge(data)
end
if credentials
if project_import_data.credentials
# nothing
else
project_import_data.credentials = {}
end
project_import_data.credentials =
project_import_data.credentials.merge(credentials)
end
project_import_data
end
end
The number of if statements and branches here makes it easy to make
mistakes. To resolve this, we refactor this code in such a way that we
can get rid of all but the first `if data.nil? && credentials.nil?`
statement. We can do this by simply sending `to_h` to `nil` in the right
places, which removes the need for statements such as `if data`.
Since this data gets written to a database, in ProjectImportData we do
make sure to not write empty Hash values. This requires an `unless`
(which is really a `if !`), but the resulting code is still very easy to
read.
|
|
Migrate emoji_menu_spec.js to Jest
See merge request gitlab-org/gitlab-ce!23688
|
|
Fix MR pipelines run on only: regexp
Closes #55026
See merge request gitlab-org/gitlab-ce!23657
|
|
|
|
|
|
|
|
|
|
Allow GitHub imports via token even if OAuth2 provider not configured
Closes #38524
See merge request gitlab-org/gitlab-ce!23703
|
|
|
|
It adds a base class for CompareTestReportsService
containing common code with CompareLicenseManagementReportsService
which is present in GitLab Enterprise Edition.
|
|
|
|
Fix some N+1 queries related to Admin Dashboard, User Dashboards and Activity Stream
Closes #55106
See merge request gitlab-org/gitlab-ce!23034
|