| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
| |
* Replace `require` or `require_relative` with `require_dependency`
* Remove unneeded `autoload`
|
| |
|
|\
| |
| |
| |
| |
| |
| | |
Fix snippets embed form
Closes #46948
See merge request gitlab-org/gitlab-ce!19219
|
|/ |
|
|\
| |
| |
| |
| |
| |
| | |
Make ActiveRecordSubscriber rails 5 compatible
Closes #44702
See merge request gitlab-org/gitlab-ce!19276
|
| | |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Add merge requests list endpoint for groups
Closes #45082
See merge request gitlab-org/gitlab-ce!18915
|
|/ / |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Fixes redudant script failure message
Closes #46552 and #44271
See merge request gitlab-org/gitlab-ce!19138
|
| | |
| | |
| | |
| | | |
So the 'failed' message can actually indicate if the build failed or not
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Script failure message was redundant so it was removed. Also 'check your
job log' message was removed from all the error messages.
Closes #44271
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
Take two for MR metrics population background migration
See merge request gitlab-org/gitlab-ce!19097
|
| | | | |
|
| | | | |
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
Add yarn clean step, to delete production compiled assets
See merge request gitlab-org/gitlab-ce!19212
|
| | | | | |
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Fix active tab styling
Closes #46799
See merge request gitlab-org/gitlab-ce!19264
|
|/ / / / / |
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Add link to hack for styling Capybara HTML dumps
See merge request gitlab-org/gitlab-ce!19205
|
|/ / / / / |
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Export assigned issues in iCalendar feed
Closes #44184
See merge request gitlab-org/gitlab-ce!17783
|
|/ / / / / |
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Eliminate cached N+1 queries for projects in Issue API
See merge request gitlab-org/gitlab-ce!19269
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
In CE, every `Issue` entity is also a `ProjectEntity`, which calls
`entity&.project.try(:id)` to show the project ID. In an API
request with 100 issues, this would hit the Rails statement cache 100 times
for the same project and cause unnecessary overhead as related models would
also be loaded.
In EE, we call `Issue#supports_weight?` for each issue, which then calls
`project&.feature_available?(:issue_weights)`. If the project is not
preloaded, this incurs additional overhead, as each individual Project
object has to be queried. This can lead to a significant performance hit.
In loading the CE project with 100 issues, this contributed to at least 22%
of the load time.
See https://gitlab.com/gitlab-org/gitlab-ce/issues/47031 for why testing this
is a bit tricky.
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Use strings as properties key in kubernetes service spec.
See merge request gitlab-org/gitlab-ce!19265
|
| | | | | | | |
|
|\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Add /vendor/gitaly-ruby to gitignore
See merge request gitlab-org/gitlab-ce!19261
|
| |/ / / / / / |
|
|\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Replace .having with .where in calendar query
Closes #46964
See merge request gitlab-org/gitlab-ce!19230
|
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
the current syntax doesn't work properly in Rails 5, the resulting query
looks like:
HAVING "events"."project_id" IN (0)
instead of:
HAVING "events"."project_id" IN (SELECT "projects"."id" FROM...
Also we should not use ActiveRecord internal methods. In this case we
can filter projects in WHERE clause instead of doing this in HAVING
clause. Usage of WHERE should be also more efficient because grouping
is then done on much smaller subset of records.
|
|\ \ \ \ \ \ \ \
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Replace `.exists` with `EXISTS ()`
Closes #46277
See merge request gitlab-org/gitlab-ce!19178
|
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
`.exists` should not be used because it's an internal ActiveRecord
method, but we can easily generate the same sql query with `EXISTS`.
|
|\ \ \ \ \ \ \ \ \
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
Use Gitaly 0.104.0
See merge request gitlab-org/gitlab-ce!19258
|
| | |_|/ / / / / /
| |/| | | | | | | |
|
|\ \ \ \ \ \ \ \ \
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
Fix encoding bug in Gitaly::Commit::TreeEntry
Closes #46445 and gitaly#1208
See merge request gitlab-org/gitlab-ce!19243
|
| |/ / / / / / / / |
|
|\ \ \ \ \ \ \ \ \
| |_|_|_|_|_|_|_|/
|/| | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
'45520-remove-links-from-web-ide-unexpectedly-navigate-you-to-a-different-page' into 'master'
Resolve "Remove links from Web IDE unexpectedly navigate you to a different page"
Closes #45520
See merge request gitlab-org/gitlab-ce!19129
|
|/ / / / / / / /
| | | | | | | |
| | | | | | | |
| | | | | | | | |
page"
|
|\ \ \ \ \ \ \ \
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Cleanup code for General Project Settings
See merge request gitlab-org/gitlab-ce!19245
|
|/ / / / / / / / |
|
|\ \ \ \ \ \ \ \
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
into 'master'
Adjust permitted params filtering on merge scheduling
Closes #46903
See merge request gitlab-org/gitlab-ce!19255
|
| |/ / / / / / / |
|
|\ \ \ \ \ \ \ \
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
'46991-unable-to-start-rails-console-if-using-master-from-gitlab-shell' into 'master'
Resolve "Unable to start rails console if using master from gitlab-shell"
Closes #46991
See merge request gitlab-org/gitlab-ce!19251
|
| | | | | | | | | |
|
|\ \ \ \ \ \ \ \ \
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
Fix cattr_accessor definition
Closes #46282
See merge request gitlab-org/gitlab-ce!19192
|
| | |_|/ / / / / /
| |/| | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Use cattr_accessor directly inside the defined class, otherwise
in Rails 5 this accessor returns nil.
|
|\ \ \ \ \ \ \ \ \
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
Removal of API v3 from the codebase
Closes #36819
See merge request gitlab-org/gitlab-ce!18970
|
| | |_|/ / / / / /
| |/| | | | | | | |
|
|\ \ \ \ \ \ \ \ \
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
Resolve "Extra whitespace when hovering commit sha in MR widget"
Closes #46971
See merge request gitlab-org/gitlab-ce!19267
|
| |\ \ \ \ \ \ \ \ \
| | | |_|_|_|/ / / /
| | |/| | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
* master: (32 commits)
Update CHANGELOG.md for 10.8.3
Update "col-xs" usage to bootstrap4
Add missing usage_ping_enabled to API settings
Make Repository#blob_data_at a public method
Fix UI broken in line profiling modal due to Bootstrap 4
Only preload member records for the relevant projects/groups/user in projects API
Fix after sign-in with Google button
Fix a wrong link reference in docs
Simplify spec: `2.times.each` -> `2.times`
Replace grape-route-helpers with our own grape-path-helpers
Add archived param to the json response
Consider we might have prefix for stable branches
Convert spec/javascripts/.eslintrc to yml format as well
Remove version lock on awesome_print dependency
Fix import project by url layout
Fix missing timeout value in object storage pre-authorization call
fix last lints
disable no-multi-assign
fix `spaced-comment`
remove unused createComponentWithMixin from vue spec helpers
...
|