Skip to content
Snippets Groups Projects
Select Git revision
  • master default protected
  • 12-9-stable
  • 12-7-stable
  • 12-6-stable
  • 12-8-stable
  • github/fork/Kloppi313/patch-1
  • 12-5-stable
  • 12-4-stable
  • github/fork/ramalokesh8477/master
  • 12-1-stable
  • 12-2-stable
  • 12-0-stable
  • 12-3-stable
  • 42-42-stable
  • github/fork/hussamgit398/patch-2
  • 12-3-auto-deploy-20190911
  • 12-3-auto-deploy-20190916
  • 12-3-auto-deploy-20190908
  • 12-3-auto-deploy-20190901
  • 12-3-auto-deploy-20190901-32664
  • v12.10.0.pre
  • v12.9.0
  • v12.9.0-rc42
  • v12.8.7
  • v12.8.6
  • v12.8.5
  • v12.8.4
  • v12.8.3
  • v12.6.8
  • v12.7.7
  • v12.8.2
  • v12.8.1
  • v12.9.0.pre
  • v12.8.0
  • v12.8.0-rc42
  • v12.5.10
  • v12.7.6
  • v12.6.7
  • v12.7.5
  • v12.5.9
40 results

cycle_analytics.rb

  • Timothy Andrew's avatar
    918e589c
    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
    History
    Implement a second round of review comments from @DouweM.
    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.