| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |
|
|
|
| |
This adds tooltips to the event target for events like "opened merge
request !XYZ"
|
| |\
| |
| |
| |
| |
| |
| |
| |
| | |
Enable Style/SpaceBeforeComma rubocop cop
No spaces before commas.
See #17478
See merge request !4360
|
| | |
| |
| |
| |
| |
| | |
No spaces before commas.
See #17478
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Enable Style/NegatedIf Rubocop cop
Favor `unless` over `if` for negative conditions (or control flow ||).
```ruby
# bad
do_something if !some_condition
# bad
do_something if not some_condition
# good
do_something unless some_condition
# good
some_condition || do_something
```
See #17478
See merge request !4355
|
| | |/
| |
| |
| |
| |
| |
| | |
Favor `unless` over `if` for negative conditions
(or control flow ||).
See #17478
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Enable Lint/LiteralInCondition rubocop cop
Detects literals used in conditions.
See #17478
See merge request !4354
|
| | |/
| |
| |
| |
| |
| | |
Checks of literals used in conditions.
See #17478
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Enable Style/MultilineIfThen rubocop cop
Do not use then for multi-line if.
See #7478.
See merge request !4353
|
| | |/
| |
| |
| |
| |
| | |
Do not use then for multi-line if.
See #7478.
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Enable Style/MethodDefParentheses rubocop cop
Use def with parentheses when there are parameters.
See #17478
See merge request !4352
|
| | |/
| |
| |
| |
| |
| | |
Use def with parentheses when there are parameters.
See #17478
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Enable Style/WhileUntilDo rubocop style cop
Checks for redundant `do` after `while` or `until`.
See #17478
See merge request !4350
|
| | |/
| |
| |
| | |
See #17478
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Enable Style/EndOfLine Rubocop style cop
Use Unix-style line endings.
Removed carriage return character from end of each line in `app/controllers/projects/find_file_controller.rb`
See #17478
See merge request !4347
|
| | |/
| |
| |
| | |
Use Unix-style line endings. See #17478.
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Enable Lint/BlockAlignment Rubocop cop
See #17478
See merge request !4346
|
| | |/
| |
| |
| | |
See #17478
|
| |/
|
|
| |
See #17478
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
|
|
| |
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/15577
|
| | |
|
| | |
|
| |
|
|
| |
fixes #17508
|
| |\
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Update notes immediately after closing/reopening
## What does this MR do?
Fixes MR https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/4127 Thanks @bendyorke!
## Are there points in the code the reviewer needs to double check?
Nope.
## Why was this MR needed?
Constant conflicts in that MR
## What are the relevant issue numbers?
## Screenshots (if relevant)
Fixes #15091
See merge request !4323
|
| | | |
|
| |/ |
|
| |\
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Fixed issue with spacing in issuable header
## What does this MR do?
Fixes an issue with spacing next to the word 'by' in the issuable header
## What are the relevant issue numbers?
Closes #17864
## Screenshots (if relevant)
See merge request !4314
|
| | |
| |
| |
| |
| |
| | |
The Ruby was returning the HTML without a spacing next to the word 'by'
Closes #17864
|
| |\ \
| |/
|/|
| |
| |
| |
| |
| |
| | |
Fixed potential issue with 2 ci status polling events happening
Possible cause for double notifications was if the request was slow & then you changed page whilst this request was happening it would finish on another page & then launch another interval - this stops that issue.
Also passed in the CI status as an option value rather than waiting for the first ajax request to finish
See merge request !3869
|
| | | |
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Filter issuables by any user
Closes #2836
See merge request !3820
|
| | | | |
|
| | | |
| | |
| | |
| | | |
Closes #2836
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Fix bug where fade div covers up controls button
## What does this MR do?
Fixes bug where fade div covers up controls button
## Screenshots (if relevant)
Before:
<img src="/uploads/a6a883839255b8e10d872759a1aa54a0/Screen_Shot_2016-05-26_at_2.22.08_PM.png" width="300px">
After:
<img src="/uploads/9ab53d9815470cf86f68f5a340ffc454/Screen_Shot_2016-05-26_at_2.22.42_PM.png" width="300px">
See merge request !4306
|
| | | | | |
|
| |\ \ \ \
| |/ / /
|/| | |
| | | |
| | | |
| | | |
| | | | |
Remove header title from project pages
With new navigation design it makes no sense to duplicate title in the header.
See merge request !4304
|
| | | | |
| | | |
| | | |
| | | | |
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
With new navigation design it makes no sense to duplicate title in the
header
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Reduced number of buttons in commit header
## What does this MR do?
Reduces the buttons in the commit header.
## What are the relevant issue numbers?
Closes #17576
## Screenshots
See merge request !4302
|
| | | | | | |
|
| | | |/ /
| |/| |
| | | |
| | | | |
Closes #17576
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Loads move issue dropdown async
To keep the style of the dropdown the same as the other dropdowns in the issue form, it uses select2 rather than our new dropdowns.
Closes #16563
See merge request !4160
|
| | | | | | |
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | | |
Closes #16563
|
| |\ \ \ \ \
| |_|_|/ /
|/| | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Updated services UI
See merge request !4177
|
| | | | | | |
|
