1. 18 Feb, 2016 8 commits
    • Yorick Peterse's avatar
      Added specs for Repository#exists? · 8a7aad77
      Yorick Peterse authored
      8a7aad77
    • Yorick Peterse's avatar
      Fixed Repository#exists? to handle errors · 54aa0969
      Yorick Peterse authored
      Now that Repository#raw_repository no longer sets the autocrlf option it
      will also no longer raise any NoRepository errors since it doesn't
      access Rugged any more. This also means that Repository#exists? can't
      simply return the raw repository as this is no indication of whether or
      not the repository actually exists (besides returning a non boolean is
      weird in the first place).
      
      To solve this problem Repository#exists? now properly checks if the
      repository exists and returns true/false instead of a
      Gitlab::Git::Repository or nil object.
      54aa0969
    • Yorick Peterse's avatar
      Only set autocrlf when creating/updating files · c475b171
      Yorick Peterse authored
      Setting the "autocrlf" Git option is an overkill since it's rarely
      actually needed. More importantly, it has quite the impact on
      performance (see gitlab-org/gitlab-ce#13457 for more information).
      
      By setting "autocrlf" when creating or updating files we guarantee the
      option is always set properly when we actually need it _without_
      introducing overhead for requests that have nothing to do with this
      option.
      
      Fixes gitlab-org/gitlab-ce#13457
      c475b171
    • Rémy Coutable's avatar
      Merge branch 're-add-missing-part-in-update-doc' into 'master' · 8b918267
      Rémy Coutable authored
      Re-add section about NGINX config and init script updates in 8.4->8.5 update doc
      
      These sections were removed by !2766 but:
      - even if the NGINX config wasn't modified, it might
        be in future updates so it's better to always have
        it instead of having to remember to add it depending
        on the changes
      - the init script update section must be there since
        it's a safe command line that should be run on every update
      
      This fixes https://github.com/gitlabhq/gitlabhq/issues/10031.
      
      /cc @rspeicher @yorickpeterse 
      
      See merge request !2815
      8b918267
    • Douwe Maan's avatar
      Merge branch 'fix/gitpushservice-complexity-issue' into 'master' · 52a8d1f3
      Douwe Maan authored
      Reduce code complexity on GitPushService#execute
      
      Code complexity for gitlab-ce after this has been refactored:
      ```
        27.3: GitPushService#execute
      ```
      This still needs to be merged into `gitlab-ee` presumably with conflicts... Perhaps we should create another issue for doing that?
      
      I left the code sort of similar to what it was... If I could, I would have refactored most of the code into separate classes, etc. as it breaks probably all SOLID principles.
      
      Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/13327
      
      
      See merge request !2784
      52a8d1f3
    • Dmitriy Zaporozhets's avatar
      Merge branch 'rs-update-unicorn' into 'master' · 27b732ab
      Dmitriy Zaporozhets authored
      Bump unicorn to `~> 4.9.0`
      
      Closes #2820 
      
      See merge request !2843
      27b732ab
    • Robert Speicher's avatar
      Merge branch 'issue_13539' into 'master' · 90a6783d
      Robert Speicher authored
      Reopened MRs should also be considered as open.
      
      Fixes #13539 
      
      See merge request !2866
      90a6783d
    • Rubén Dávila's avatar
      2770de96
  2. 17 Feb, 2016 32 commits