summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Grapify the service APIgrapify-service-apiRobert Schilling2016-12-122-7/+17
|
* Grapify the service APIRobert Schilling2016-12-0911-85/+700
|
* Merge branch 'fix/rename-mwbs-to-merge-when-pipeline-succeeds' into 'master' Kamil Trzciński2016-12-0524-104/+132
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rename MWBS service to Merge When Pipeline Succeeds ## What does this MR do? This MR renames Merge When Build Succeeds feature to Merge When Pipeline Succeeds, because we recently changed the behavior, see https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6675. ## Does this MR meet the acceptance criteria? - [x] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG.md) entry added - [x] [Documentation created/updated](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/doc/development/doc_styleguide.md) - [x] All builds are passing ## What are the relevant issue numbers? Closes #23354 See merge request !7135
| * Fix Rubocop offense in merge request specsfix/rename-mwbs-to-merge-when-pipeline-succeedsGrzegorz Bizon2016-11-301-1/+0
| |
| * Merge branch 'master' into fix/rename-mwbs-to-merge-when-pipeline-succeedsGrzegorz Bizon2016-11-30311-2250/+3323
| |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * master: (110 commits) Rewrite an HTTP link to use HTTPS Edit /spec/features/profiles/preferences_spec.rb to match changes in 084d90ac Add blue back to sub nav active Remove JSX/React eslint plugins. Fix a transient spec failure Adds hoverstates for collapsed Issue/Merge Request sidebar Moved groups above projects Add StackProf to the Gemfile, along with a utility to get a profile for a spec Update Sidekiq-cron to fix compatibility issues with Sidekiq 4.2.1 Add a CHANGELOG entry Alert user when logged in user email is not the same as the invitation Expose timestamp in build entity used by serializer Rename `MergeRequest#pipeline` to `head_pipeline` Remove unnecessary database indexes CE-specific changes gitlab-org/gitlab-ee#1137 Fixing typo & Clarifying Key name fix started_at check fix blob controller spec failure - updated not to use file-path- fix blob controller spec failure Merge branch 'jej-use-issuable-finder-instead-of-access-check' into 'security' ... Conflicts: app/controllers/projects/merge_requests_controller.rb lib/api/merge_requests.rb spec/requests/api/merge_requests_spec.rb
| * | Move MWPS document to new locationAchilleas Pipinellis2016-11-296-49/+54
| | |
| * | Fix tests for allowing merged if pipeline succeededGrzegorz Bizon2016-11-251-2/+2
| | |
| * | Merge branch 'master' into fix/rename-mwbs-to-merge-when-pipeline-succeedsGrzegorz Bizon2016-11-25963-4137/+8668
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * master: (312 commits) Fix bad selection on dropdown menu for tags filter Fixed issue boards scrolling with a lot of lists & issues You can only assign default_branch when editing a project ... Don't convert data which already is the target type Stop supporting Google and Azure as backup strategies renames some of the specs and adds changelog entry Fixed dragging issue moving wrong issue after multiple drags of issue Fixed issue boards issue sorting when dragging issue into list Rephrase some system notes to be compatible with new system note style Add missing JIRA file that redirects to the new location Fix documentation to create the `pg_trm` extension before creating the DB Document that we always use `do...end` for `before` in RSpec Backport Note#commands_changes from EE Log mv_namespace parameters Add default_branch attr to Project API payload in docs. Fix title case to sentence case properly escape username validation error message flash Remove header ids from University docs Add missing documentation. Added test that checks the correct select box is there for the LFS ... ... Conflicts: app/services/system_note_service.rb spec/features/merge_requests/merge_when_pipeline_succeeds_spec.rb spec/services/merge_requests/merge_when_pipeline_succeeds_service_spec.rb spec/services/system_note_service_spec.rb
| * | | Add Changelog entry for Merge When Pipeline SucceedsGrzegorz Bizon2016-11-211-0/+4
| | | |
| * | | Update some docs to reflect MWPS name changeGrzegorz Bizon2016-11-2110-24/+24
| | | |
| * | | Rename MWPS in user interface and feature testsGrzegorz Bizon2016-11-212-15/+16
| | | |
| * | | Rename MWPS in system notes and related testsGrzegorz Bizon2016-11-215-8/+8
| | | |
| * | | Rename MWBS service to Merge When Pipeline SucceedsGrzegorz Bizon2016-11-217-17/+34
| | | |
* | | | Merge branch 'zj-issue-new-over-issue-create' into 'master' Kamil Trzciński2016-12-053-2/+13
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Accept `issue new` as command to create an issue Checkout commit message: 629eb073c2056d0e4d87aefea83d5d384465bee6 cc @pedroms See merge request !7787
| * | | | Accept `issue new` as command to create an issueZ.J. van de Weg2016-12-053-2/+13
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now only `/trigger issue create` is a valid command, but our UI shows Issue new everywhere. The default now will be `/trigger issue new`. The help message is adjusted to reflect this. Fixes: gitlab-org/gitlab-ce#25025
* | | | Merge branch '25148_fix_gfm_typo' into 'master' Achilleas Pipinellis2016-12-041-2/+2
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix gfm doc typo about two spaces for next line transfer Fixes #25148 See merge request !7915
| * | | | fix gfm doc typo about two spaces for next line transferRydkin Maxim2016-12-031-2/+2
| | | | |
* | | | | Merge branch 'api-expose-commiter-details' into 'master' Rémy Coutable2016-12-044-2/+20
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | API: Expose committer details for a commit Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/22312 See merge request !7849
| * | | | API: Expose committer details for a commitapi-expose-commiter-detailsRobert Schilling2016-12-014-2/+20
| | | | |
* | | | | Merge branch 'improve-patch-update-guide' into 'master' Achilleas Pipinellis2016-12-031-2/+5
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Patch upgrade documentation should be consulted on stable branch Closes #25245. See merge request !7892
| * | | | | Patch upgrade documentation should be consulted on stable branchimprove-patch-update-guideRémy Coutable2016-12-021-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [ci skip] Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | | | | Merge branch 'revert-918d30bf' into 'master' Achilleas Pipinellis2016-12-031-6/+0
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Revert "Merge branch 'docs/patch-guide-link-to-old-guide' into 'master'" Reverting in favor of https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/7892 See merge request !7911
| * | | | | | Revert "Merge branch 'docs/patch-guide-link-to-old-guide' into 'master'"Achilleas Pipinellis2016-12-031-6/+0
|/ / / / / / | | | | | | | | | | | | This reverts merge request !7909
* | | | | | Merge branch 'docs/patch-guide-link-to-old-guide' into 'master' Achilleas Pipinellis2016-12-031-0/+6
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Link to old patch update guide now that we install workhorse with a raketask Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/25284 See merge request !7909
| * | | | | | Link to old patch update guide now that we install workhorse with a raketaskAchilleas Pipinellis2016-12-031-0/+6
|/ / / / / / | | | | | | | | | | | | | | | | | | [ci skip]
* | | | | | Merge branch '23696-fix-diff-view-highlighting' into 'master' Fatih Acet2016-12-039-577/+568
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Resolve "Highlighting lines is broken" ## What does this MR do? Add line highlighting back to diff view. This was working in the MR "changes" tab, but not on a commit page such as https://gitlab.com/winniehell/reproduction-area/commit/9101e66f5761929002956e0f2dd65d7f8643903d ~~This MR also fixes the `scrollToElement` method in `MergeRequestTabs` to account for the extra height of the tab links which are now fixed in place once they are scrolled to the top of the screen.~~ (removed in favor of !7051) This MR also refactors much of the `Diff` and `MergeRequestTabs` classes to es6 syntax in an effort to increase readability. ## Are there points in the code the reviewer needs to double check? Check out both MR "change" tabs and commit diff pages and ensure that line highlighting works and that loading a page with one of these permalink hashes correctly highlights and scrolls to the line. Ensure I didn't break anything in the transition to es6 syntax. Check the functionality of the tabs on MR pages, as well as diff page interactivity (unfolding hidden lines in diff files, adding comments to diffs, etc). I have checked these myself, but another set of eyes would be a good idea. ## Does this MR meet the acceptance criteria? - [x] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG.md) entry added - Tests - [x] All builds are passing - [x] Conform by the [merge request performance guides](http://docs.gitlab.com/ce/development/merge_request_performance_guidelines.html) - [x] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides) - [x] Branch has no merge conflicts with `master` (if it does - rebase it please) - [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits) ## What are the relevant issue numbers? Closes #23696 See merge request !7090
| * | | | | | satisfy eslint no-param-reassign rule23696-fix-diff-view-highlightingMike Greiling2016-12-022-8/+10
| | | | | | |
| * | | | | | display error when a tab cannot be loadedMike Greiling2016-12-021-1/+5
| | | | | | |
| * | | | | | use shorthand instead of length > 0 ↵Mike Greiling2016-12-021-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | (http://airbnb.io/javascript/#comparison--shortcuts)
| * | | | | | use e instead of event variable nameMike Greiling2016-12-022-10/+10
| | | | | | |
| * | | | | | adjust scrollToElement to account for fixed merge request tabsMike Greiling2016-12-021-5/+7
| | | | | | |
| * | | | | | re-apply MR !6285 "Added forceLoad ability to singleFileDiffs, added ↵Luke Bennett2016-12-023-10/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | callback to getContentHTML, added conditional force load if a collapsed diff line anchor is found" Use url utility to retrieve hash
| * | | | | | fix eslint failures on Diff and MergeRequestTabsMike Greiling2016-12-023-117/+119
| | | | | | |
| * | | | | | remove array destructuring until we can fix babel configMike Greiling2016-12-021-2/+6
| | | | | | |
| * | | | | | refactor Diff class event bindingsMike Greiling2016-12-021-48/+48
| | | | | | |
| * | | | | | refactor Diff to es6 class syntaxMike Greiling2016-12-023-42/+35
| | | | | | |
| * | | | | | prefer es6 string interpolation over concatinationMike Greiling2016-12-021-27/+26
| | | | | | |
| * | | | | | refactor awkward bind workarounds into arrow functionsMike Greiling2016-12-021-69/+44
| | | | | | |
| * | | | | | refactor MergeRequestTabs to es6 class syntaxMike Greiling2016-12-023-91/+88
| | | | | | |
| * | | | | | update CHANGELOG.md to reflect changes in !7090Mike Greiling2016-12-021-0/+4
| | | | | | |
| * | | | | | fix diff line highlighting by moving method from the MergeRequestTabs class ↵Mike Greiling2016-12-022-42/+48
| | | | | | | | | | | | | | | | | | | | | | | | | | | | to the Diff class.
| * | | | | | temporarily revert "Added forceLoad ability to singleFileDiffs, added ↵Mike Greiling2016-12-022-28/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | callback to getContentHTML, added conditional force load if a collapsed diff line anchor is found" This reverts commit d2ee380816fa161d94da54c1f7e594c9a2ba2241.
* | | | | | | Merge branch '24921-hide-prompt-to-add-ssh-key-if-ssh-protocol-is-disabled' ↵Sean McGivern2016-12-033-1/+22
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | into 'master' If SSH prototol is disabled don't say the user requires SSH keys Closes #24921 See merge request !7840
| * | | | | | Test all values for `enabled_git_access_protocol`Andrew Smith2016-12-021-9/+14
| | | | | | |
| * | | | | | If SSH prototol is disabled don't say the user requires SSH keysAndrew Smith2016-12-013-1/+17
| | | | | | |
* | | | | | | Merge branch 'system-font-test' into 'master' Jacob Schatz2016-12-0228-140/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use system font Just for fun.... ### macOS Chrome (San Francisco): ![Screen_Shot_2016-11-17_at_4.01.29_PM](/uploads/bfbab440ea0136314a6a7cffcd44f3ff/Screen_Shot_2016-11-17_at_4.01.29_PM.png) ### macOS Safari (San Francisco): ![Screen_Shot_2016-11-18_at_8.28.22_AM](/uploads/3adc05d7adf84ca726cbaaf36cb0567d/Screen_Shot_2016-11-18_at_8.28.22_AM.png) #### Ubuntu Firefox (Ubuntu): ![Screen_Shot_2016-11-18_at_8.31.08_AM](/uploads/0f0d581d2d79d5db3dba9b2200499dc7/Screen_Shot_2016-11-18_at_8.31.08_AM.png) #### Windows 7 IE 11 (Segoe UI): ![Screen_Shot_2016-11-18_at_8.35.05_AM](/uploads/88efc2c554252de03fb84cc3c3e2856e/Screen_Shot_2016-11-18_at_8.35.05_AM.png) See merge request !7545
| * | | | | | | Remove source sans prosystem-font-testAnnabel Dunstone Gray2016-12-0128-143/+1
| | | | | | | |
| * | | | | | | Use system fontAnnabel Dunstone Gray2016-12-011-1/+4
| | | | | | | |
* | | | | | | | Merge branch 'mr-origin-7855' into 'master' Rémy Coutable2016-12-024-17/+7
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds a default commit message when adding a README (#25167) Closes #25167 See merge request !7903
| * | | | | | | | Added a changelog entrymattl2016-12-021-0/+4
| | | | | | | | |