Skip to content

Remove SuppressCompositePrimaryKeyWarning concern

Max Orefice requested to merge morefice/remove-primary-key-warning-concern into master

Ref: #292909 (closed)

What does this MR do and why?

This MR removes SuppressCompositePrimaryKeyWarning concern as we can explicitly set the primary key in the model directly instead to silent this warning.

➜  gitlab git:(432604-fix-project-builder-cross-joins) ✗ bundle exec rspec --color spec/models/project_authorization_spec.rb
Run options: include {:focus=>true}

All examples were filtered out; ignoring {:focus=>true}

Test environment set up in 3.35989 seconds
.................

Finished in 9.81 seconds (files took 13.9 seconds to load)
17 examples, 0 failures

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Max Orefice

Merge request reports