Read about our upcoming Code of Conduct on this issue

This instance will be upgraded to Heptapod 0.28.1 on 2022-01-26 at 16:00 UTC+1 (a few minutes of down time)

  1. 20 Feb, 2019 1 commit
  2. 11 Feb, 2019 1 commit
  3. 08 Feb, 2019 2 commits
  4. 06 Feb, 2019 1 commit
  5. 31 Jan, 2019 2 commits
  6. 29 Jan, 2019 1 commit
    • 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.
  7. 28 Jan, 2019 1 commit
  8. 25 Jan, 2019 1 commit
    • Gabriel Mazetto's avatar
      Refactor Storage Migration · ab819a7a1396
      Gabriel Mazetto authored
      Specs were reviewed and improved to better cover the current behavior.
      There was some standardization done as well to facilitate the
      implementation of the rollback functionality.
      StorageMigratorWorker was extracted to HashedStorage namespace were
      RollbackerWorker will live one as well.
  9. 24 Jan, 2019 1 commit
  10. 21 Jan, 2019 1 commit
  11. 16 Jan, 2019 1 commit
    • 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 1 commit
  13. 11 Jan, 2019 2 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).
    • Reuben Pereira's avatar
      Add table and model for error tracking settings · 56c4cbadc13e
      Reuben Pereira authored
    • Steve Azzopardi's avatar
      Add explicit test for #latest_successful_build_for! · adbd6b49ae38
      Steve Azzopardi authored
      project.latest_successful_build_for! is being tested inside of `describe
      explicit. In doing so some duplication was generated but not
      `#latest_successful_build_for!` has full coverage unlike before.
      Move `create_pipeline` & `create_build` as helper methods for this spec
      to reduce duplication.
    • 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 2 commits
  21. 13 Dec, 2018 1 commit
  22. 12 Dec, 2018 1 commit
  23. 08 Dec, 2018 5 commits