1. 03 Aug, 2016 9 commits
    • Kirill Smelkov's avatar
      NXD Teach GitLab about patches · 2a771bd3
      Kirill Smelkov authored
      Teach GitLab not only to merge changes from a merge-request, but also to
      apply patches posted to merge-request in a way like `git am` would do -
      without merge commit and directly on top of current branch. Which way to
      go is selected by user in web UI, and apply patches is the first option.
      
      There are 3 cases:
      
      - only 1 commit is present in MR -> the only available option is to
        apply that single commit as one patch without a merge
      
        ( There is no need for merge commit in this case at all: information
          about user who applied the patch goes to "Committer" field in resultant
          commit. Avoiding 1 merge per 1 patch results in cleaner history )
      
        It is also possible to review patch description directly in web UI,
        before doing the actual application, and correct / amend it as needed.
      
      - several commits are present in MR:
      
        * it is possible to apply the patches directly on top of current
          branch. Again information about who applied what goes to "Committer"
          field.
      
        * it is possible to merge MR changes with making a merge commit.
      
          This variant is useful, when patches from a MR do several logical
          steps to reach one goal, and MR description contain cover letter for
          whole patch series.
      
          in this case original commits stay untouched and resulting merge
          will contain MR author as author, user who accepted MR as committer,
          and cover letter as merge commit message.
      
          NOTE we avoid useless "Merge branch X into Y" in merge message, and
              just put MR title into merge subject and MR description into merge
              description.
      
              This way it is more logical with more important information in
              merge subject and thus e.g. more handy to oversee what a merge brings,
              just by it subject, e.g. via looking at updates via
      
                  gitk --first-parent ...
      
              or via web.
      
      NOTE for pre-generated references to merge-request we now use full MR
          URL, instead of !<MR-n>. Full URLs work everywhere, not only on
          original site where MR was created, or even only in original repo
          and not its fork on the same site.
      2a771bd3
    • Kirill Smelkov's avatar
      NXD Show ICP on front page & on footer of every other pages · 9cb9b232
      Kirill Smelkov authored
      TODO detect whether request comes from China and only then show ICP (?).
      9cb9b232
    • Kirill Smelkov's avatar
      NXD Show "about" footer on every page · 374c1d83
      Kirill Smelkov authored
      We show in small font size the same info that is shown on sign_in page:
      
          "GitLab Nexedi Edition", "About GitLab" and "About Nexedi"
      
      This is good to have and hereby-introduced about-footer area will be
      also used in the next patch for ICP too.
      
      XXX placement of .about-footer to be near bottom is done not very
      correctly.
      374c1d83
    • Kirill Smelkov's avatar
      NXD gitlab:app:check : Don't check for init script · c7e1f952
      Kirill Smelkov authored
      Like Omnibus, SlapOS version does not have init script - nothing to
      check here.
      c7e1f952
    • Kirill Smelkov's avatar
      NXD lib/tasks/gitlab/check: Exit with non-zero code, if something failed in a check task · dc0b40da
      Kirill Smelkov authored
      This is handy for monitoring tools, which could e.g. periodically call check
      tasks and instead of parsing output, rely on exit code.
      
      The way we detect if something failed is via hooking into String#red, and if
      anything was ever printed in red - that's an error.
      dc0b40da
    • Kirill Smelkov's avatar
    • Kirill Smelkov's avatar
      NXD clone_panel: Remove SSH option completely · b52ad92d
      Kirill Smelkov authored
      The default was switched to HTTP in the previous patch, but let's completely
      remove SSH option - we support only HTTP for git fetch/push.
      b52ad92d
    • Kirill Smelkov's avatar
      NXD Make HTTP to be the default clone protocol · 7f890211
      Kirill Smelkov authored
      Both fetch and push are possible over https, which is selected by http if
      gitlab was configured to use https in external url.
      
      This way to reduce security vectors and possible ways to interact with gitlab
      we use https only without ssh at all.
      7f890211
    • Kirill Smelkov's avatar
      NXD GitLab Nexedi Edition · 29f7a210
      Kirill Smelkov authored
      = GitLab Community Edition + Nexedi patches
      29f7a210
  2. 30 Jun, 2016 3 commits
  3. 28 Jun, 2016 1 commit
  4. 27 Jun, 2016 2 commits
  5. 15 Jun, 2016 1 commit
  6. 14 Jun, 2016 4 commits
  7. 20 May, 2016 1 commit
  8. 19 May, 2016 5 commits
  9. 12 May, 2016 6 commits
    • Yorick Peterse's avatar
      Version 8.7.5 · 0e8b7d8e
      Yorick Peterse authored
      0e8b7d8e
    • Yorick Peterse's avatar
      Moved !3886 changelog entry to 8.7.5 · ec447ee7
      Yorick Peterse authored
      ec447ee7
    • Robert Speicher's avatar
      Merge branch '15529-fix=missing-from-clause-for-table-issues' into 'master' · 2d154e54
      Robert Speicher authored
      Fix an issue when filtering merge requests with more than one label
      
      Fixes #15529.
      
      See merge request !3886
      2d154e54
    • Yorick Peterse's avatar
      Updated CHANGELOG for 8.7.5 · 872a0934
      Yorick Peterse authored
      872a0934
    • Stan Hu's avatar
      Merge branch 'fix-build-notification-on-merge-page-change' into 'master' · 70b630d2
      Stan Hu authored
      Fix build notification on merge request page change even if the build status didn't change
      
      ## What does this MR do?
      This MR contains a bugfix for #17357 which was introduced by !3998. The notification are now only shown on status changes, and not when switching between different merge requests.
      
      ## Are there points in the code the reviewer needs to double check?
      Check implementation
      
      ## Why was this MR needed?
      Because auf a bug introduced in !3998.
      
      ## What are the relevant issue numbers?
      #17357
      
      Closes #17357
      
      See merge request !4086
      70b630d2
    • Rémy Coutable's avatar
      Merge branch '16568-relative-links-wiki-regression' into 'master' · e181dfc2
      Rémy Coutable authored
      Relative Links in the Wiki Are Broken
      
      - [ ] #16568 (!4050) Relative links in wiki are broken
          - [x] Investigate issue
      
        
          - [x] Implementation / Fix
      
              - [x] Write (failing) tests for `WikiLinkFilter`
              - [x] Link to `./bar` should either get rewritten correctly or left alone
              - [x] Link to `./bar.md` should maybe get rewritten correctly (is left alone currently)
              - [x] Link to `bar.md` should get rewritten correctly
      
      
                  - [x] Check if this indeed a bug
      
          - [x] Make sure CI is green
          - [x] Assign to endboss
      
          - [x] Wait for review
      
          - [x] Implement review feedback
          - [ ] Wait for merge
      
      
      See merge request !4050
      e181dfc2
  10. 11 May, 2016 8 commits