Skip to content

Move summarize_my_code_review feature flag to ee/

What does this MR do and why?

  • This MR moves the summarize_my_code_review feature flag from config/feature_flags/development/summarize_my_code_review.yml to ee/config/feature_flags/development/summarize_my_code_review.yml since summarize code review feature is only accessible in EE

Background

  • This feature flag was initially introduced by this MR: !118807 (merged)
  • Later on, this another MR was merged to consolidate the AI feature authorization logic in one class Llm::FeatureAuthorizer: !137211 (merged). And hence, moving push_frontend_feature_flag(:summarize_my_code_review, current_user) to ee/ is no longer valid: !118807 (comment 1372626322)

Testing

🟡 Local AI features setup is still pending for me but this change is relatively safe and covered via unit test cases in ee/spec/lib/gitlab/duo/developments/setup_spec.rb and ee/spec/policies/merge_request_policy_spec.rb

Related to #409136 (closed)

Edited by Kinshuk Singh

Merge request reports

Loading