| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
27762-add-default-artifacts-expiration
* upstream/master: (247 commits)
Switched CONTRIBUTING.md style guide recommendation for method chaining
Fix new offenses
Stylistic tweaks
Fix OAuth/SAML user blocking behavior
Revert "Enable Style/DotPosition"
Revert "Prefer leading style for Style/DotPosition"
Revert "Enable Style/BarePercentLiterals"
Manually correct autocorrect
Move up delegate calls
Exclude migrations from Style/MutableConstant
ActiveSupport delegation is preferred over Forwardable
Update haml_lint to work with newest rubocop
Add explanations to cops
Update rubocop and rubocop-rspec and regenerate .rubocop_todo.yml
Update rubocop and rubocop-rspec and regenerate .rubocop_todo.yml
Order cops alphabetically
Don’t exclude some file in lib from rubocop
Fix new offenses
Enable Rails/Delegate
Enable Style/WordArray
...
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This reverts commit e00fb2bdc2090e9cabeb1eb35a2672a882cc96e9.
# Conflicts:
# .rubocop.yml
# .rubocop_todo.yml
# lib/gitlab/ci/config/entry/global.rb
# lib/gitlab/ci/config/entry/jobs.rb
# spec/lib/gitlab/ci/config/entry/factory_spec.rb
# spec/lib/gitlab/ci/config/entry/global_spec.rb
# spec/lib/gitlab/ci/config/entry/job_spec.rb
# spec/lib/gitlab/ci/status/build/factory_spec.rb
# spec/lib/gitlab/incoming_email_spec.rb
|
| |
| |
| |
| | |
This reverts commit cb10b725c8929b8b4460f89c9d96c773af39ba6b.
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| |\
| | |
| | |
| | |
| | | |
Grapify the CI::Runners API
See merge request !9440
|
| | | |
|
| |/ |
|
| | |
|
| |
| |
| |
| |
| |
| | |
C: Style/MultilineOperationIndentation:
Align the operands of an expression in an assignment
spanning multiple lines.
|
| |
| |
| |
| |
| |
| |
| |
| | |
Feedback:
* https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/9219#note_23343951
* https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/9219#note_23344036
* https://gitlab.com/gitlab-org/gitlab-ce/issues/27762#note_23344797
|
|/
|
|
| |
TODO: Add tests and screenshots
|
|\
| |
| |
| |
| |
| |
| | |
Add ability to define a coverage regex in the .gitlab-ci.yml
Closes #20428
See merge request !7447
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Instead of using:
`coverage: /\(\d+.\d+%\) covered/`
This structure must be used now:
```
coverage:
output_filter: /\(\d+.\d+%\) covered/`
```
The surrounding '/' is optional.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This commit:
* Turns `coverage_regex` into `coverage` entry in yml file
* Fixes smaller requests from code reviewers for the previous commit
* This commit is temporary (will be squashed afterwards)
This commit does not (further commits will do though):
* Add global `coverage` entry handling in yml file as suggested by Grzegorz
* Add specs
* Create changelog
* Create docs
|
| |
| |
| |
| | |
* Instead of using the proposed `coverage` key, this expects `coverage_regex`
|
| | |
|
| | |
|
| |
| |
| |
| |
| | |
The conflict happens when we try to update a build,
but fail to do so due to fact that we update the same build concurrently for two different runners.
|
| | |
|
|/
|
|
| |
https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/1083
|
|\ |
|
| |\
| | |
| | |
| | |
| | |
| | |
| | | |
Fix Double Spaced CI Log
Closes #19086
See merge request !8349
|
| | | |
|
| |/
| |
| |
| |
| |
| | |
Fixes #26051.
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
| |
| |
| |
| | |
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
| |\
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Give forbidden if project for the build was deleted
I guess we don't need a change log entry because this is just for an internal corner case fix.
Closes #25309
See merge request !8091
|
| | |
| | |
| | |
| | |
| | | |
Feedback:
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/8091#note_20253762
|
| | |
| | |
| | |
| | |
| | | |
Feedback:
https://gitlab.com/gitlab-org/gitlab-ce/builds/7606797
|
| | |
| | |
| | |
| | |
| | | |
Feedback:
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/8091#note_20222666
|
| | |
| | |
| | |
| | | |
Closes #25309
|
| | |
| | |
| | |
| | |
| | | |
Feedback:
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/8088#note_20235080
|
| |/
| |
| |
| |
| |
| |
| | |
So that this would be more consistent with the other variables,
which all of them are string based.
Closes #25554
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
* We realized that headers were not set whenever we give 204
because `render_api_error!` doesn't preserve the headers.
* We also realized that `update_runner_info` would be called in
POST /builds/register every time therefore runner is updated
every time, ticking the queue, making this last_update didn't
work very well, and the test would be failing due to that.
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
|/ |
|
| |
|
| |
|
| |
|
| |
|