1. 22 Nov, 2018 1 commit
  2. 25 Jul, 2018 1 commit
  3. 13 Jun, 2017 1 commit
  4. 23 Feb, 2017 2 commits
  5. 20 Sep, 2016 3 commits
    • Timothy Andrew's avatar
      Implement a second round of review comments from @DouweM. · 918e589c
      Timothy Andrew authored
      - Don't use `TableReferences` - using `.arel_table` is shorter!
      - Move some database-related code to `Gitlab::Database`
      - Remove the `MergeRequest#issues_closed` and
        `Issue#closed_by_merge_requests`  associations. They were either
        shadowing or were too similar to existing methods. They are not being
        used anywhere, so it's better to remove them to reduce confusion.
      - Use Rails 3-style validations
      - Index for `MergeRequest::Metrics#first_deployed_to_production_at`
      - Only include `CycleAnalyticsHelpers::TestGeneration` for specs that
        need it.
      - Other minor refactorings.
      918e589c
    • Timothy Andrew's avatar
      Fix `ON DELETE CASCADE` migrations. · 8747f29d
      Timothy Andrew authored
      Incorrect syntax.
      8747f29d
    • Timothy Andrew's avatar
      Implement review comments from @yorickpeterse · 8957293d
      Timothy Andrew authored
      1. Change multiple updates to a single `update_all`
      
      2. Use cascading deletes
      
      3. Extract an average function for the database median.
      
      4. Move database median to `lib/gitlab/database`
      
      5. Use `delete_all` instead of `destroy_all`
      
      6. Minor refactoring
      8957293d
  6. 19 Sep, 2016 1 commit
    • Timothy Andrew's avatar
      Remove unused merge request metrics. · 2cddd02e
      Timothy Andrew authored
      - These are not being used anymore.
      - Consolidate all issue metrics into a single migration.
      - Consolidate all merge request metrics into a single migration.
      2cddd02e
  7. 26 Aug, 2016 3 commits
  8. 04 Aug, 2016 1 commit
  9. 01 Aug, 2016 1 commit
  10. 16 Jun, 2016 2 commits
  11. 09 Jun, 2016 1 commit
    • Sean McGivern's avatar
      Enable RuboCop for migrations · 98bb435f
      Sean McGivern authored
      Migrations shouldn't fail RuboCop checks - especially lint checks, such
      as the nested method check. To avoid changing code in existing
      migrations, add the magic comment to the top of each of them to skip
      that file.
      98bb435f
  12. 03 Jun, 2016 2 commits
  13. 25 May, 2016 1 commit
  14. 19 May, 2016 2 commits
  15. 12 May, 2016 2 commits