Age | Commit message (Collapse) | Author | |
---|---|---|---|
2018-07-16 | Enable frozen string in apps/uploaders/*.rb | gfyoung | |
Partially addresses #47424. | |||
2018-02-02 | use Gitlab::UserSettings directly as a singleton instead of ↵ | Mario de la Ossa | |
including/extending it | |||
2017-08-31 | `current_application_settings` belongs on `Gitlab::CurrentSettings` | Sean McGivern | |
The initializers including this were doing so at the top level, so every object loaded after them had a `current_application_settings` method. However, if someone had rack-attack enabled (which was loaded before these initializers), it would try to load the API, and fail, because `Gitlab::CurrentSettings` didn't have that method. To fix this: 1. Don't include `Gitlab::CurrentSettings` at the top level. We do not need `Object.new.current_application_settings` to work. 2. Make `Gitlab::CurrentSettings` explicitly `extend self`, as we already use it like that in several places. 3. Change the initializers to use that new form. | |||
2017-08-29 | replace `is_spam?` with `spam?` | Maxim Rydkin | |
2017-08-14 | Whitelist or fix additional `Gitlab/PublicSend` cop violations | Robert Speicher | |
An upcoming update to rubocop-gitlab-security added additional violations. | |||
2017-05-10 | Enable the Style/TrailingCommaInLiteral cop | Rémy Coutable | |
Use the EnforcedStyleForMultiline: no_comma option. Signed-off-by: Rémy Coutable <remy@rymai.me> | |||
2016-09-20 | Reduce duplication in AkismetService | Robert Speicher | |
2016-08-16 | Further refactor and syntax fixes. | Patricio Cano | |
2016-08-15 | Refactored AkismetHelper into AkismetService and cleaned up `Spammable` | Patricio Cano | |
- Refactored SpamCheckService into SpamService |