- 12 May, 2016 4 commits
-
-
Andrei Gliga authored
-
Andrei Gliga authored
-
Timothy Andrew authored
- Separate 'exercise' and 'verify' steps of tests. - Use `build_stubbed` instead of `build`
-
Timothy Andrew authored
- And fix behavior for non-file hierarchical links.
-
- 11 May, 2016 2 commits
-
-
Sean McGivern authored
The repository push email can be very expensive to generate, especially with syntax-highlighted diffs. Instead of generating the email for each recipient, generate one email object and reset the Message-Id and To headers for each recipient. (Cloning would also be expensive in the case of large emails, although probably not as bad as generating from scratch.)
-
Sean McGivern authored
-
- 10 May, 2016 7 commits
-
-
Sean McGivern authored
`User#starred_projects` doesn't perform any visibility checks. This has a couple of problems: 1. It assumes a user can always view all of their starred projects in perpetuity (project not changed to private, access revoked, etc.). 2. It assumes that we'll only ever allow a user to star a project they can view. This is currently the case, but bugs happen. Add `User#viewable_starred_projects` to filter the starred projects by those the user either has explicit access to, or are public or internal. Then use that in all places where we list the user's starred projects.
-
Rémy Coutable authored
Signed-off-by: Rémy Coutable <remy@rymai.me>
-
Grzegorz Bizon authored
-
Grzegorz Bizon authored
Closes #17342, closes #17363
-
Rémy Coutable authored
- Fix naming of API::CommitStatuses - Ensure we use require_dependency instead of require - Ensure the namespace is right in lib/api/api.rb, otherwise, we might require Grape::API::Helpers which defines the `#params` method. This is to avoid requiring a file multiple times and getting an "Already initialized constant" error. Signed-off-by: Rémy Coutable <remy@rymai.me>
-
Rémy Coutable authored
Signed-off-by: Rémy Coutable <remy@rymai.me>
-
Rémy Coutable authored
This reverts commit 4cc85a58, reversing changes made to 90ae445b. Signed-off-by: Rémy Coutable <remy@rymai.me>
-
- 09 May, 2016 6 commits
-
-
Stan Hu authored
Closes #1625
-
Jeroen van Baarsen authored
In 8278b763 the default behaviour of annotation has changes, which was causing a lot of noise in diffs. We decided in #17382 that it is better to get rid of the whole annotate gem, and instead let people look at schema.rb for the columns in a table. Fixes: #17382
-
Rémy Coutable authored
Signed-off-by: Rémy Coutable <remy@rymai.me>
-
cnam-dep authored
-
Jacob Vosmaer authored
-
Artem Sidorenko authored
-
- 08 May, 2016 4 commits
-
-
Robert Speicher authored
-
Benedikt Huss authored
-
Long Nguyen authored
-
Long Nguyen authored
-
- 07 May, 2016 2 commits
-
-
Stan Hu authored
Closes #17348
-
Stan Hu authored
Use a custom Loofah scrubber since sanitize 2.x transformers are inadequate to handle case-sensitive SVG attributes. sanitize parses documents as HTML instead of XML, which causes all SVG attribute names (e.g. viewBox) to be downcased. * SVG element list: https://www.w3.org/TR/SVG/eltindex.html * SVG attribute list: https://www.w3.org/TR/SVG/attindex.html Closes #14555
-
- 06 May, 2016 5 commits
-
-
Ahmad Sherif authored
Fixes #14002
-
Sean McGivern authored
-
Sean McGivern authored
If a branch starts with an issue's IID, followed by a hyphen, the description will be updated to say that is closes the issue. This also updates the title of the merge request to 'Resolves "$issue-title"', as long as: - There is more than one commit in the merge request (if there is only one commit, the commit's title will be used as before) - The issue's IID is valid for the project
-
Sean McGivern authored
-
Takuya Noguchi authored
-
- 05 May, 2016 4 commits
-
-
Felipe Artur authored
-
Douglas Barbosa Alexandre authored
Pull Request Review Comments are comments on a portion of the unified diff.
-
Long Nguyen authored
-
Long Nguyen authored
-
- 04 May, 2016 6 commits
-
-
DJ Mountney authored
Each test reloads the trusted_proxies initializer, which in turn will set Rails.application.config.action_dispatch.trusted_proxies to something new. This will leak into the other tests, but the middleware that it is used in has already been loaded for the whole test suite, so it should have no impact.
-
Benedikt Huss authored
-
Benedikt Huss authored
-
Annabel Dunstone authored
-
Annabel Dunstone authored
-
Stan Hu authored
Closes #17243
-