Age | Commit message (Collapse) | Author |
|
|
|
|
|
This cop prevents you from using file in API, it points you to the
development documentation about workhorse file acceleration.
|
|
rails_helper.rb's only logic was to require
spec_helper.rb.
|
|
CE-EE parity for root files
See merge request gitlab-org/gitlab-ce!32287
|
|
|
|
Use Rubocop's Include
instead of manually checking the matcher in controllers specs.
|
|
|
|
Stop checking the file location programmatically.
|
|
|
|
Moves every fixture source file from
spec/javascripts to spec/frontend
|
|
Related issue:
https://gitlab.com/gitlab-org/gitlab-ee/issues/6565
|
|
The `#reload` makes to load all objects into memory,
and the main purpose of `#reload` is to drop the association cache.
The `#reset` seems to solve exactly that case.
|
|
We also enable the rubocop that makes it mandatory
|
|
|
|
This style change enforces `return if ...` instead of
`return nil if ...` to save maintainers a few minor review points
|
|
This change allows the GitLab rails and sidekiq components to receive
tracing spans from upstream services such as Workhorse and pass these
spans on to downstream services including Gitaly and Sidekiq.
This change will also emit traces for incoming and outgoing requests
using the propagated trace information. This will allow operators and
engineers to view traces across the Workhorse, GitLab Rails, Sidekiq and
Gitaly components.
Additional intra-service instrumentation will be added in future
changes.
|
|
Updates specs to use new rails5 format.
The old format:
`get :show, { some: params }, { some: headers }`
The new format:
`get :show, params: { some: params }, headers: { some: headers }`
|
|
This Cop enforces the rule that injecting EE modules (using prepend,
include, or extend) is done by placing the injection on the last line of
a file, instead of somewhere in the middle. By placing these lines at
the very end, merge conflicts will not happen.
|
|
This should wrap up frozen string
enablement for gitlab-ce.
xref gitlab-org/gitlab-ce#47424.
|
|
E2E Test - LDAP login with SSL
Closes gitlab-qa#59
See merge request gitlab-org/gitlab-ce!22574
|
|
|
|
|
|
|
|
Enable even more frozen string in lib/**/*.rb
See merge request gitlab-org/gitlab-ce!22206
|
|
Use ACCEPT_INSECURE_CERTS env var and fix step
Simplify saml signin
Fix rubo cop offence
Add missing # frozen_string_literal: true
|
|
Moves RSpec/FilePath cop from gitlab-styles, so we can specify
filepaths to ignore without hardcoding them in the gem.
|
|
Enables frozen string for the following files:
* lib/generators/**/*.rb
* lib/gitaly/**/*.rb
* lib/google_api/**/*.rb
* lib/haml_lint/**/*.rb
* lib/json_web_token/**/*.rb
* lib/mattermost/**/*.rb
* lib/microsoft_teams/**/*.rb
* lib/object_storage/**/*.rb
* lib/omni_auth/**/*.rb
* lib/peek/**/*.rb
* lib/rouge/**/*.rb
* lib/rspec_flaky/**/*.rb
* lib/system_check/**/*.rb
Partially addresses #47424.
|
|
|
|
|
|
These are dynamic translations, so has to be marked explicitly using `N_`,
but they are not used in runtime, so can exist in separate file.
https://github.com/grosser/gettext_i18n_rails#unfound-translations-with-rake-gettextfind
|
|
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
|
|
|
Server Side Request Forgery in Services and Web Hooks
See merge request gitlab/gitlabhq!2337
|
|
|
|
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
|
|
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
|
|
|
Also try to use the same config from EE
|
|
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
|
|
|
And use .rubocop.yml to exclude paths we don't care,
rather than using the cop itself to exclude.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
'master'
Decrease Metrics/CyclomaticComplexity threshold to 15
See merge request !13905
|