| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
| |
Includes compatibility with Rails 5. We don’t have any tests for this,
so no need to run them. [ci skip]
Changelog: https://github.com/ctran/annotate_models/releases/tag/v2.7.0
Working towards #14286.
|
|
|
|
|
|
| |
Sprockets 3.5.0 reintroduces GZIP-ing and 3.6.0 includes performance improvements. Changelog: https://github.com/rails/sprockets/blob/3.x/CHANGELOG.md
Resolves #14344.
|
|\
| |
| |
| |
| | |
Bump rails to 4.2.6
See merge request !3424
|
| | |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Redis connection pool
Split from https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/3232
Having an easily accessible Redis connection pool allows us to do more
cool stuff with Redis in GitLab (instead of having to go through e.g.
the Rails cache).
See merge request !3521
|
| |/ |
|
|/
|
|
|
|
|
|
|
|
|
|
| |
This removes a few dependencies! It was also rude to be using coveralls
0.8.9, considering 0.8.12 introduced support for GitLab CI :) Also
paves the way for updating mime-types to 3.0.
Coveralls Changelog:
https://github.com/lemurheavy/coveralls-ruby/releases
Simplecov Changelog:
https://github.com/colszowka/simplecov/blob/master/CHANGELOG.md
|
|
|
|
|
|
|
|
|
|
| |
This was originally suggested by @ayufan and modified to be a bit
cleaner and use RequestStore instead of a regular Hash.
By caching the output of the two methods involved the number of queries
is reduced significantly. For example, for an issue with 200 notes (of
which 100 reference a number of merge requests) this cuts down the
amount of queries from around 6300 to around 3300.
|
|\
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Upgrade Brakeman from 3.1.4 to 3.2.1
Includes support for Rails 5, bug fixes, and various additional vulnerability checks.
[See Changelog for more information](https://github.com/presidentbeef/brakeman/blob/master/CHANGES).
Working towards #14286.
See merge request !3397
|
| |
| |
| |
| |
| |
| |
| |
| | |
Includes support for Rails 5, bug fixes, and various additional
vulnerability checks.
See Changelog for more information
(https://github.com/presidentbeef/brakeman/blob/master/CHANGES)
|
|/
|
|
|
|
| |
Includes compatibility with Rails 5.
See the changelog for more information (https://github.com/modeset/teaspoon/blob/master/CHANGELOG.md).
|
|\
| |
| |
| |
| |
| |
| | |
Backport premailer and diff changes from EE
Backport part of https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/151
See merge request !3318
|
| |
| |
| |
| |
| | |
# Conflicts:
# Gemfile.lock
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Discussed in #14233.
See [their releases](https://github.com/bbatsov/rubocop/releases) for
more info.
Changes:
- Enable DisplayCopNames for lint output.
- Default behavior for `Alias` changed, set to enforce `prefer_alias_method`.
- Enabling Rails cops changed to new syntax.
- Remove StyleGuides and move Descriptions to comments.
- Add missing cops.
- Add TODOs for cops that should be enabled in the future.
- Set TargetRubyVersion to 2.1.
|
|
|
|
|
|
|
| |
This builds on the changes introduced in
https://gitlab.com/gitlab-org/gitlab_git/merge_requests/72 and results
in merge requests with large diffs (e.g. due to them containing minified
CSS) loading much faster.
|
| |
|
| |
|
|\
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
CSS style guide
Working towards what was discussed in #13552, this adds the [SCSS Linter gem](https://github.com/brigade/scss-lint) for style guide conformity in CI.
TODO:
- [x] Agree on and write SCSS Style Guide Documentation.
- [x] Document the `scss-lint` config file.
- [x] Figure out how best to run this in CI, right now it's taking longer than I would think it should.
- [x] Use CSSComb for auto-correction (Maybe just include a CSSComb config file and have developers run the node package manually if they're interested in using it?).
My logic for not using CSSComb in the first place is that, AFAIK, we don't currently require Node/NPM, even in the dev environment. Maybe I'm wrong about that? `scss-lint` is a Ruby implementation of a similar concept, but it doesn't include autocorrect.
Is there a way we can run get GitLab CI to run `scss-lint` without having to put together the full GitLab application? Seeing as it's just static analysis, it seems like a waste of time/resources.
/cc @jschatz1 @rspeicher
See merge request !3069
|
| |
| |
| |
| |
| |
| | |
documentation for SCSS Style Guide.
See !3069 for more information.
|
| |
| |
| |
| |
| | |
This reverts commit 01160fc06182de89c400af174861f6545ad6ceb8, reversing
changes made to 4bff9daf8b6d85e9c78565e21cfaa3f6d36f0282.
|
|/ |
|
| |
|
|\
| |
| |
| |
| |
| |
| | |
Bump poltergeist to `~> 1.9.0`
See merge request !3116
|
| | |
|
| | |
|
|/ |
|
|\
| |
| |
| |
| |
| |
| | |
Upgrade carrierwave to 0.10.0.
Closes #2805
See merge request !3084
|
| | |
|
| | |
|
| | |
|
|/ |
|
|
|
|
| |
Closes #6069
|
|
|
|
|
| |
See https://groups.google.com/forum/#!msg/rubyonrails-security/ddY6HgqB2z4/we0RasMZIAAJ
and https://groups.google.com/forum/#!msg/rubyonrails-security/ly-IH-fxr_Q/WLoOhcMZIAAJ
|
|
|
|
|
|
| |
In an effort to get the JS file size down, i've removed NGProgress and replaced with TurboLinks own loading indicator
Ref #13820
|
|\ |
|
| |\
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Revert "Remove newrelic gem. Closes #12860"
This reverts commit 531767175f0a364a332c74bf27c5f202732ce519.
Conflicts:
Gemfile.lock
See merge request !2943
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This reverts commit 531767175f0a364a332c74bf27c5f202732ce519.
Conflicts:
Gemfile.lock
|
|\ \ \
| |/ / |
|
| |/ |
|
| | |
|
| |\
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Update Gemojione for new hotness :sparkles:
Before | After
------ | -----
![Screen_Shot_2016-02-12_at_3.38.43_PM](/uploads/299e5ffe38c91657085741261a678f81/Screen_Shot_2016-02-12_at_3.38.43_PM.png) | ![Screen_Shot_2016-02-12_at_3.38.09_PM](/uploads/ccee29719a6445a9efd2498391a7eea0/Screen_Shot_2016-02-12_at_3.38.09_PM.png)
See merge request !2800
|
| | | |
|
| | | |
|
|\ \ \
| |/ /
| | |
| | |
| | | |
Conflicts:
app/models/user.rb
|
| |/ |
|
| |
| |
| |
| | |
Closes #13218
|
|/ |
|
|\
| |
| |
| |
| |
| |
| |
| | |
Dedicated method for counting commits between refs
Corresponding gitlab_git merge request: https://gitlab.com/gitlab-org/gitlab_git/merge_requests/66
Issue that lead to these changes: https://gitlab.com/gitlab-org/gitlab-ce/issues/12619
See merge request !2707
|
| |
| |
| |
| |
| |
| |
| | |
gitlab_git 8.1 adds the ability to count the amount of commits between
two references without having to allocate anything but regular
Rugged::Commit objects. This in turn speeds up the process of counting
the number of commits a branch is ahead/behind by about 3.5x.
|
|\ \
| |/
|/|
| |
| |
| |
| | |
Remove newrelic gem. Closes #12860
See merge request !2682
|