Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Fix link text from group context | Jarka Kadlecova | 2017-11-23 | 1 | -6/+6 |
| | |||||
* | Batchload blobs for diff generation | Zeger-Jan van de Weg | 2017-11-21 | 1 | -2/+2 |
| | | | | | | | | | | | | | | | | | | After installing a new gem, batch-loader, a construct can be used to queue data to be fetched in bulk. The gem was also introduced in both gitlab-org/gitlab-ce!14680 and gitlab-org/gitlab-ce!14846, but those mrs are not merged yet. For the generation of diffs, both the old blob and the new blob need to be loaded. This for every file in the diff, too. Now we collect all these so we do 1 fetch. Three `.allow_n_plus_1_calls` have been removed, which I expect to be valid, but this needs to be confirmed by a full CI run. Possibly closes: - https://gitlab.com/gitlab-org/gitlab-ce/issues/37445 - https://gitlab.com/gitlab-org/gitlab-ce/issues/37599 - https://gitlab.com/gitlab-org/gitlab-ce/issues/37431 | ||||
* | Optimise getting the pipeline status of commitsci-pipeline-status-query | Yorick Peterse | 2017-11-16 | 1 | -3/+6 |
| | | | | | This adds an optimised way of getting the latest pipeline status for a list of Commit objects (or just a single one). | ||||
* | Remove unnecessary comments | Douwe Maan | 2017-09-25 | 1 | -2/+0 |
| | |||||
* | Properly compare diff refs and diff positions when shas are truncated | Douwe Maan | 2017-09-25 | 1 | -5/+6 |
| | |||||
* | Adds cacheless render to Banzai object render34509-improves-markdown-rendering-performance-for-commits-list | Tiago Botelho | 2017-09-06 | 1 | -0/+9 |
| | |||||
* | pass whole commit to Gitlab::Gpg::Commit again | Alexis Reigel | 2017-09-05 | 1 | -1/+1 |
| | | | | | we need the commit object for the updated verification that also checks the committer's email to match the gpg key and user's emails. | ||||
* | Merge branch 'sm-cherry-pick-list-commits-in-message' into 'master' | Sean McGivern | 2017-09-01 | 1 | -0/+22 |
|\ | | | | | | | | | Add 'from commit' information to cherry-picked commits See merge request !13475 | ||||
| * | Change a variable name in Commit#cherry_pick_description | Saverio Miroddi | 2017-08-21 | 1 | -4/+4 |
| | | |||||
| * | Reverse order of commits in MR cherry-pick message | Saverio Miroddi | 2017-08-21 | 1 | -1/+1 |
| | | | | | | | | Also improved the UT for better documenting this change. | ||||
| * | Correct the cherry-pick message for merge commits | Saverio Miroddi | 2017-08-17 | 1 | -10/+7 |
| | | | | | | | | | | The list of commits must be generated from the merge request, not from a diff of the branches. | ||||
| * | Add support for start project to commit cherry-pick description | Saverio Miroddi | 2017-08-15 | 1 | -4/+4 |
| | | |||||
| * | Made commit cherry-pick message more standard | Saverio Miroddi | 2017-08-15 | 1 | -11/+8 |
| | | | | | | | | | | | | Follow the `cherry pick -x` format for the first line, and the interactive rebase format for the commits in the merge, in case of merge commit | ||||
| * | Extracted cherry-pick description in ::Commit | Saverio Miroddi | 2017-08-15 | 1 | -6/+10 |
| | | | | | | | | This also fixes the inconsistency between non/merge commits in the number of newlines introduced. | ||||
| * | Add 'from commit' information to cherry-picked commits | Saverio Miroddi | 2017-08-10 | 1 | -0/+24 |
| | | | | | | | | | | | | Store the original commit in the commit message. In case of merge commit, the commits added to the start branch are also listed. | ||||
* | | Merge branch 'dm-gpg-signature-performance' into 'master' | Robert Speicher | 2017-08-16 | 1 | -1/+1 |
|\ \ | | | | | | | | | | | | | Only create commit GPG signature when necessary See merge request !13561 | ||||
| * | | Only create commit GPG signature when necessarydm-gpg-signature-performance | Douwe Maan | 2017-08-16 | 1 | -1/+1 |
| |/ | |||||
* | | Whitelist or fix additional `Gitlab/PublicSend` cop violationsrs-more-public-send-whitelists | Robert Speicher | 2017-08-14 | 1 | -1/+1 |
|/ | | | | | An upcoming update to rubocop-gitlab-security added additional violations. | ||||
* | Move `deltas` and `diff_from_parents` logic to Gitlab::Git::Commit | Alejandro Rodríguez | 2017-08-07 | 1 | -12/+2 |
| | | | | | | This helps keep the abstraction layers simpler, and also keep the interface of those methods consistent, in case of implementation changes. | ||||
* | Refactor Gitlab::Git::Commit to include a repository | Alejandro Rodríguez | 2017-08-07 | 1 | -1/+2 |
| | |||||
* | fetch gpg signature badges by ajax | Alexis Reigel | 2017-07-27 | 1 | -1/+7 |
| | |||||
* | move signature cache read to Gpg::Commit | Alexis Reigel | 2017-07-27 | 1 | -5/+0 |
| | | | | | | | | as we write the cache in the gpg commit class already the read should also happen there. This also removes all logic from the main commit class, which just proxies the call to the Gpg::Commit now. | ||||
* | bail if the commit has no signature | Alexis Reigel | 2017-07-27 | 1 | -5/+1 |
| | |||||
* | cache the gpg commit signature | Alexis Reigel | 2017-07-27 | 1 | -20/+5 |
| | | | | | we store the result of the gpg commit verification in the db because the gpg verification is an expensive operation. | ||||
* | verify gpg commit using tmp keyring and db query | Alexis Reigel | 2017-07-27 | 1 | -1/+16 |
| | |||||
* | use more descriptive variable names | Alexis Reigel | 2017-07-27 | 1 | -5/+8 |
| | |||||
* | commit signature with spec | Alexis Reigel | 2017-07-27 | 1 | -1/+1 |
| | |||||
* | Prototype key verification | Kevin O'Connor | 2017-07-27 | 1 | -0/+12 |
| | |||||
* | Merge branch 'request-store-wrap' into 'master' | Rémy Coutable | 2017-07-18 | 1 | -16/+3 |
|\ | | | | | | | | | Add RequestCache to cache via RequestStore See merge request !12920 | ||||
| * | Rename the methods to make it fit with current name | Lin Jen-Shin | 2017-07-18 | 1 | -1/+1 |
| | | |||||
| * | Follow feedback on the merge request | Lin Jen-Shin | 2017-07-18 | 1 | -1/+1 |
| | | |||||
| * | Use RequestStoreWrap for Commit#author | Lin Jen-Shin | 2017-07-18 | 1 | -16/+3 |
| | | | | | | | | | | | | | | We also try to use instance variable to cache the result if RequestStore is not available, so we could keep the same logic, using the same cache key. Also introduce a way to specify method specific cache key | ||||
* | | Renamed Gitaly services | Andrew Newdigate | 2017-07-18 | 1 | -2/+2 |
|/ | |||||
* | Add table for merge request commits | Sean McGivern | 2017-07-06 | 1 | -1/+1 |
| | | | | | | | | | | | This is an ID-less table with just three columns: an association to the merge request diff the commit belongs to, the relative order of the commit within the merge request diff, and the commit SHA itself. Previously we stored much more information about the commits, so that we could display them even when they were deleted from the repo. Since 8.0, we ensure that those commits are kept around for as long as the target repo itself is, so we don't need to duplicate that data in the database. | ||||
* | Merge branch '12614-fix-long-message' into 'master' | Dmitriy Zaporozhets | 2017-06-08 | 1 | -16/+11 |
|\ | | | | | | | | | | | | | Fix long urls in the title of commit Closes #12614 See merge request !10938 | ||||
| * | Fix long urls in the title of commit | Alexander Randa | 2017-05-12 | 1 | -16/+11 |
| | | |||||
* | | Re-enable Gitaly commit_raw_diffs migration | Ahmad Sherif | 2017-06-07 | 1 | -6/+5 |
| | | |||||
* | | Merge branch 'fix/use-new-gitaly-commit-diff-rpc' into 'master' | Robert Speicher | 2017-06-02 | 1 | -5/+6 |
|\ \ | | | | | | | | | | | | | Use the new Gitaly CommitDiff RPC See merge request !11826 | ||||
| * | | Use the new Gitaly CommitDiff RPCfix/use-new-gitaly-commit-diff-rpc | Ahmad Sherif | 2017-06-02 | 1 | -5/+6 |
| | | | |||||
* | | | Merge branch 'rc/enable-PreferredHashMethods-cop' into 'master' | Robert Speicher | 2017-06-02 | 1 | -1/+1 |
|\ \ \ | | | | | | | | | | | | | | | | | Enable the Style/PreferredHashMethods cop See merge request !11883 | ||||
| * | | | Enable the Style/PreferredHashMethods coprc/enable-PreferredHashMethods-cop | Rémy Coutable | 2017-06-02 | 1 | -1/+1 |
| |/ / | | | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me> | ||||
* | | | Lint our factories creation in addition to their build33139-lint-our-factories | Rémy Coutable | 2017-06-02 | 1 | -1/+1 |
|/ / | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me> | ||||
* | | Add support for deltas_only under Gitaly | Ahmad Sherif | 2017-05-13 | 1 | -2/+10 |
| | | | | | | | | Closes gitaly#199 | ||||
* | | Remove deltas_only from DiffCollection | Jacob Vosmaer | 2017-05-11 | 1 | -4/+1 |
| | | |||||
* | | Enable the Style/TrailingCommaInLiteral cop | Rémy Coutable | 2017-05-10 | 1 | -1/+1 |
| | | | | | | | | | | | | Use the EnforcedStyleForMultiline: no_comma option. Signed-off-by: Rémy Coutable <remy@rymai.me> | ||||
* | | Use Rails 'delegate' | Jacob Vosmaer | 2017-05-09 | 1 | -3/+1 |
| | | |||||
* | | Don't make Commit#raw_deltas private | Jacob Vosmaer | 2017-05-09 | 1 | -4/+4 |
| | | |||||
* | | Don't use DiffCollection for deltas | Jacob Vosmaer | 2017-05-09 | 1 | -1/+5 |
| | | |||||
* | | Re-enable Gitaly commit_raw_diff feature | Ahmad Sherif | 2017-05-07 | 1 | -7/+8 |
| | | |||||
* | | Rename latest_pipeline to last_pipelinealways-show-latest-pipeline-in-commit-box | Lin Jen-Shin | 2017-05-03 | 1 | -2/+2 |
| | | | | | | | | | | | | | | | | As it might be confusing that it's not the same as Pipeline#latest Feedback: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/11038#note_28677090 |