Read about our upcoming Code of Conduct on this issue

  1. 18 Feb, 2019 1 commit
  2. 11 Feb, 2019 1 commit
  3. 08 Feb, 2019 1 commit
  4. 06 Feb, 2019 2 commits
  5. 04 Feb, 2019 2 commits
  6. 01 Feb, 2019 1 commit
  7. 31 Jan, 2019 3 commits
  8. 29 Jan, 2019 2 commits
    • Adrian Moisey's avatar
      Add GitLab Pages predefined variables · 857be4e36e6b
      Adrian Moisey authored
    • Andreas Brandl's avatar
      Flush InternalId records after import · 08cbb3d84cef
      Andreas Brandl authored
      After the import has finished, we flush (delete) the InternalId records
      related to the project at hand. This means we're starting over with
      tracking correct internal id values, a new record will be created
      automatically when the next internal id is generated.
      The GitHub importer assigns iid values by using supplied values from
      GitHub. We skip tracking internal id values during the import in favor
      of higher concurrency. Deleting any InternalId records after the import
      has finished is an extra measure to guarantee consistency.
  9. 28 Jan, 2019 1 commit
  10. 21 Jan, 2019 1 commit
  11. 16 Jan, 2019 2 commits
    • Yorick Peterse's avatar
      Refactor code for protecting default branches · 8ff5f72d9a09
      Yorick Peterse authored
      This refactors some of the logic used for protecting default branches,
      in particular Project#after_create_default_branch. The logic for this
      method is moved into a separate service class. Ideally we'd get rid of
      Project#after_create_default_branch entirely, but unfortunately
      Project#after_import depends on it. This means it has to stick around
      until we also refactor Project#after_import.
      For branch protection levels we introduce
      Gitlab::Access::BranchProtection, which provides a small wrapper around
      Integer based branch protection levels. Using this class removes the
      need for having to constantly refer to Gitlab::Access::PROTECTION_*
    • Yorick Peterse's avatar
      Refactor checking personal project limits · da8b222b69e8
      Yorick Peterse authored
      This refactors the code used for checking if a user has exceeded the
      personal projects limit. As part of this refactor the method has been
      renamed from Project#check_limit to "check_personal_projects_limit", as
      this name makes it much more clear what the purpose of the method is.
      Standalone unit tests have also been added, as before we only had a
      single generic validation test that did not cover all cases.
      The old implementation of the refactored method also included a `rescue`
      statement. This code would only run when a project creator was not set.
      The error that would be added wasn't super useful, especially since
      there would already be errors for the creator not being present. As none
      of the other code in the "check_personal_projects_limit" raises, it has
      been removed.
  12. 14 Jan, 2019 2 commits
  13. 11 Jan, 2019 3 commits
  14. 09 Jan, 2019 1 commit
    • Steve Azzopardi's avatar
      Remove get_build method for find_by_id · 2f9c04015f7b
      Steve Azzopardi authored
      The original intention of `get_build` was as a workaround not to violate
      `CodeReuse/ActiveRecord`. `find_by_id` does the exact same thing but
      does not violate the rubocop rule.
  15. 07 Jan, 2019 5 commits
    • Gabriel Mazetto's avatar
      Only set as `read_only` when starting the per-project migration · 10d9a7b13e9f
      Gabriel Mazetto authored
      In the previous code, we locked the project during the migration
      scheduling step, which works fine for small setups, but can be
      problematic in really big installations.
      We now moved the logic to inside the worker, so we minimize the time a
      project will be read-only. We also make sure we only do that if
      reference counter is `0` (no current operation is in progress).
    • Peter Leitzen's avatar
      Implement error tracking configuration · 6334ce10dfe7
      Peter Leitzen authored
      Re-use operations controller which already handles tracing settings.
    • Reuben Pereira's avatar
      Add table and model for error tracking settings · 56c4cbadc13e
      Reuben Pereira authored
    • Steve Azzopardi's avatar
      Create `get_build` for project model · 4f7020aeb1af
      Steve Azzopardi authored
      Inside of `Projects::ArtifactsController` and
      `Projects::BuildArtifactsController` we fetching the build by id using
      active record directly which violates `CodeReuse/ActiveRecord` rubocop
      rule. Create `get_build` inside of `project` model which does the same
    • Steve Azzopardi's avatar
      Refactor project.latest_successful_builds_for def · 50a1dd6a79ff
      Steve Azzopardi authored
      `project.latest_successful_builds_for(ref)` is being used to find a
      single job all the time. This results into us having to call `find_by`
      inside of the controller which violates our CodeReuse/ActiveRecord
      rubocop rule.
      Refactor `project.latest_successful_builds_for(ref)` to
      `project.latest_successful_build_for(job_name, ref)` which will execute
      the `find_by` inside of the model.
      Also create `project.latest_successful_build_for!(job_name, ref)` which
      raises an exception instead of returning nil.
  16. 04 Jan, 2019 2 commits
  17. 03 Jan, 2019 3 commits
  18. 02 Jan, 2019 3 commits
  19. 21 Dec, 2018 1 commit
  20. 19 Dec, 2018 3 commits