| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
| |
probably due to recent changes in `UploadsController`.
|
| |
|
| |
|
| |
|
|
|
|
|
| |
the ci status icons are generated client side, wo we don't need the
static files anymore.
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
the initial reason for this change was that graphicsmagick does not
support writing to ico files. this fact lead to a chain of changes:
1. use png instead of ico (browser support is good enough)
2. render the overlays on the client using the canvas API. this way we
only need to store the original favion and generate the overlay versions
dynamically.
this change also enables (next step) to simplify the handling of the
stock favicons as well, as we don't need to generate all the versions
upfront.
|
| |
|
|
|
|
|
|
|
|
|
| |
this ways we can keep the `lib/gitlab/ci/status/*` classes to return the
bare favicon name as it was before.
also the favicon uploader versions are now have the same names as the
stock favicons (+ `favicon_` prefix), which makes working with the
status names easier.
|
| |
|
| |
|
|
|
|
|
|
|
|
| |
Without the '?' at the end of the favicon url the custom favicon (i.e.
the favicons that are served through `UploadController`) are not shown
in the browser. It may have something to do with how `#send_file` /
`#send_data` set http headers. When serving the same icon file from the
public directory everything is fine.
|
| |
|
|
|
|
| |
`#send_file` won't properly set the favicon's content type otherwise.
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
| |
when versions have a different file format from the original file
carrierwave constructs a wrong url (with the original file suffix).
|
| |
|
| |
|
|\
| |
| |
| |
| | |
Add Configuration team to the list
See merge request gitlab-org/gitlab-ce!19416
|
| | |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Prevent Gitaly WriteConfig log noise
Closes gitaly#1221
See merge request gitlab-org/gitlab-ce!19375
|
|/ / |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Add background migrations to archive legacy job traces
Closes #46642
See merge request gitlab-org/gitlab-ce!19194
|
| | | |
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | | |
migration's.
|
| | | |
|
| | | |
|
| | | |
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
Also verify if extending would override a class method
See merge request gitlab-org/gitlab-ce!19377
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Since extending a class means including on the singleton class of the
class, this should now complain this:
``` ruby
module M
extend Gitlab::Utils::Override
override :f
def f
super.succ
end
end
class C
extend M
def self.f
0
end
end
```
It should complain because `C.f` wasn't calling `M#f`.
This should pass verification:
``` ruby
module M
extend Gitlab::Utils::Override
override :f
def f
super.succ
end
end
class B
def self.f
0
end
end
class C < B
extend M
end
```
Because `C.f` would now call `M#f`, and `M#f` does override something.
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
'46452-nomethoderror-undefined-method-previous_changes-for-nil-nilclass' into 'master'
Resolve "NoMethodError: undefined method `previous_changes' for nil:NilClass"
Closes #46452
See merge request gitlab-org/gitlab-ce!19190
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Perform pull request IO work outside a transaction
See merge request gitlab-org/gitlab-ce!19372
|
| | |/ / /
| |/| | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
When importing a GitHub pull request we would perform all work in a
single database transaction. This is less than ideal, because we perform
various slow Git operations when creating a merge request. This in turn
can lead to many DB connections being used, while just waiting for an IO
operation to complete.
To work around this, we now move most of the heavy lifting out of the
database transaction. Some extra error handling is added to ensure we
can resume importing a partially imported pull request, instead of just
throwing an error.
This commit also changes the specs for IssueImporter so they don't rely
on deprecated RSpec methods.
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
'master'
Update security products job and artifact names in documentation.
See merge request gitlab-org/gitlab-ce!19315
|
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
gitlab-org/gitlab-ee#6184
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Support rails5 in postgres indexes function and fix some migrations
Closes #46236
See merge request gitlab-org/gitlab-ce!19400
|
| | | | | | | |
|
|\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Fix Rubocop offense in build specs
See merge request gitlab-org/gitlab-ce!19419
|
|/ / / / / / / |
|
|\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Optimise pages service calling
See merge request gitlab-org/gitlab-ce!19388
|