Add feature categories for group-related controllers

This is a bit more of a commit message to make Danger happy. These are
like the other initial feature categories for controllers: a best
effort, and not guaranteed to be accurate.

app/controllers/groups/avatars_controller.rb

@@ -5,6 +5,8 @@ class Groups::AvatarsController < Groups::ApplicationController
   skip_cross_project_access_check :destroy
+  feature_category :subgroups
+
   def destroy
     @group.remove_avatar!
     @group.save

app/controllers/groups/boards_controller.rb

@@ -12,6 +12,8 @@ class Groups::BoardsController < Groups::ApplicationController
     push_frontend_feature_flag(:boards_with_swimlanes, group, default_enabled: false)
   end
+  feature_category :boards
+
   private
   def assign_endpoint_vars

app/controllers/groups/children_controller.rb

@@ -5,6 +5,8 @@ class ChildrenController < Groups::ApplicationController
     before_action :group
     skip_cross_project_access_check :index
+    feature_category :subgroups
+
     def index
       parent = if params[:parent_id].present?
                  GroupFinder.new(current_user).execute(id: params[:parent_id])

app/controllers/groups/deploy_tokens_controller.rb

@@ -3,6 +3,8 @@
 class Groups::DeployTokensController < Groups::ApplicationController
   before_action :authorize_destroy_deploy_token!
+  feature_category :continuous_delivery
+
   def revoke
     @token = @group.deploy_tokens.find(params[:id])
     @token.revoke!

app/controllers/groups/group_links_controller.rb

@@ -4,6 +4,8 @@ class Groups::GroupLinksController < Groups::ApplicationController
   before_action :authorize_admin_group!
   before_action :group_link, only: [:update, :destroy]
+  feature_category :subgroups
+
   def create
     shared_with_group = Group.find(params[:shared_with_group_id]) if params[:shared_with_group_id].present?

app/controllers/groups/group_members_controller.rb

@@ -19,6 +19,8 @@ def self.admin_not_required_endpoints
                                   :approve_access_request, :leave, :resend_invite,
                                   :override
+  feature_category :authentication_and_authorization
+
   def index
     @sort = params[:sort].presence || sort_value_name

app/controllers/groups/imports_controller.rb

@@ -3,6 +3,8 @@
 class Groups::ImportsController < Groups::ApplicationController
   include ContinueParams
+  feature_category :importers
+
   def show
     if @group.import_state.nil? || @group.import_state.finished?
       if continue_params[:to]

app/controllers/groups/labels_controller.rb

@@ -10,6 +10,8 @@ class Groups::LabelsController < Groups::ApplicationController
   respond_to :html
+  feature_category :issue_tracking
+
   def index
     respond_to do |format|
       format.html do

app/controllers/groups/milestones_controller.rb

@@ -9,6 +9,8 @@ class Groups::MilestonesController < Groups::ApplicationController
     push_frontend_feature_flag(:burnup_charts, @group)
   end
+  feature_category :issue_tracking
+
   def index
     respond_to do |format|
       format.html do

app/controllers/groups/packages_controller.rb

@@ -4,6 +4,8 @@ module Groups
   class PackagesController < Groups::ApplicationController
     before_action :verify_packages_enabled!
+    feature_category :package_registry
+
     private
     def verify_packages_enabled!

app/controllers/groups/registry/repositories_controller.rb

@@ -7,6 +7,8 @@ class RepositoriesController < Groups::ApplicationController
       before_action :verify_container_registry_enabled!
       before_action :authorize_read_container_image!
+      feature_category :package_registry
+
       def index
         respond_to do |format|
           format.html

app/controllers/groups/releases_controller.rb

@@ -2,6 +2,8 @@
 module Groups
   class ReleasesController < Groups::ApplicationController
+    feature_category :release_evidence
+
     def index
       respond_to do |format|
         format.json do

app/controllers/groups/runners_controller.rb

@@ -7,6 +7,8 @@ class Groups::RunnersController < Groups::ApplicationController
   before_action :runner, only: [:edit, :update, :destroy, :pause, :resume, :show]
+  feature_category :continuous_integration
+
   def show
     render 'shared/runners/show'
   end

app/controllers/groups/settings/ci_cd_controller.rb

@@ -13,6 +13,8 @@ class CiCdController < Groups::ApplicationController
       end
       before_action :define_variables, only: [:show]
+      feature_category :continuous_integration
+
       NUMBER_OF_RUNNERS_PER_PAGE = 4
       def show

app/controllers/groups/settings/integrations_controller.rb

@@ -7,6 +7,8 @@ class IntegrationsController < Groups::ApplicationController
       before_action :authorize_admin_group!
+      feature_category :integrations
+
       def index
         @integrations = Service.find_or_initialize_all(Service.for_group(group)).sort_by(&:title)
       end

app/controllers/groups/settings/repository_controller.rb

@@ -10,6 +10,8 @@ class RepositoryController < Groups::ApplicationController
         push_frontend_feature_flag(:ajax_new_deploy_token, @group)
       end
+      feature_category :continuous_delivery
+
       def create_deploy_token
         result = Groups::DeployTokens::CreateService.new(@group, current_user, deploy_token_params).execute
         @new_deploy_token = result[:deploy_token]

app/controllers/groups/shared_projects_controller.rb

@@ -6,6 +6,8 @@ class SharedProjectsController < Groups::ApplicationController
     before_action :group
     skip_cross_project_access_check :index
+    feature_category :subgroups
+
     def index
       shared_projects = GroupProjectsFinder.new(
         group: group,

app/controllers/groups/uploads_controller.rb

@@ -9,6 +9,8 @@ class Groups::UploadsController < Groups::ApplicationController
   before_action :authorize_upload_file!, only: [:create, :authorize]
   before_action :verify_workhorse_api!, only: [:authorize]
+  feature_category :subgroups
+
   private
   def upload_model_class

app/controllers/groups/variables_controller.rb

@@ -6,6 +6,8 @@ class VariablesController < Groups::ApplicationController
     skip_cross_project_access_check :show, :update
+    feature_category :continuous_integration
+
     def show
       respond_to do |format|
         format.json do

app/controllers/groups_controller.rb

@@ -46,6 +46,17 @@ class GroupsController < Groups::ApplicationController
   layout :determine_layout
+  feature_category :subgroups, [
+                     :index, :new, :create, :show, :edit, :update,
+                     :destroy, :details, :transfer
+                   ]
+
+  feature_category :audit_events, [:activity]
+  feature_category :issue_tracking, [:issues, :issues_calendar, :preview_markdown]
+  feature_category :code_review, [:merge_requests]
+  feature_category :projects, [:projects]
+  feature_category :importers, [:export, :download_export]
+
   def index
     redirect_to(current_user ? dashboard_groups_path : explore_groups_path)
   end