Changelog: added

* Before the name "GitLab Duo" existed, we called Duo Chat "Tanuki Bot".
  As a result, there is a lot of code in gitlab-org/gitlab that references
  this old name.
* The code here was using the old name and is no longer used
* https://gitlab.com/gitlab-org/gitlab/-/issues/431311
* https://gitlab.com/gitlab-org/gitlab/-/issues/431310

- this method was never intended to be called outside of the module,
  but has been. So we'll add it here to encourage proper use
  as documented in https://docs.gitlab.com/ee/development/ee_features.html#saas-only-feature

Inspired by the ActiveRecord migration classes

Contributes to https://gitlab.com/gitlab-org/gitlab/-/issues/421591

Follow-up for
https://gitlab.com/gitlab-org/gitlab/-/merge_requests/132111

* Remove `Gitlab::RuggedInstrumentation`
* Remove `Peek::Views::Rugged`
* Remove lint-rugged script
* Adjust tests and documentation

Changelog: removed

- move the check to the left a bit and allow developer
  to detect the issue quicker before danger.

These methods don't have retry in their implementation
and require it from outside.

MR: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/132639

Changelog: other

Adds support code to start requiring milestones as well as including the
current milestone when generating a new migration.

Update some failing tests

Slight updates

Add `namespaces` and `projects to the list of forbidden tables,
in order to recude the risk of LWLock saturation.

See
https://gitlab.com/gitlab-com/gl-infra/scalability/-/issues/2353#note_1606203744.

- better conformity to current hash styles

Changelog: changed

MR: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/133436

Changelog: added

Unite RSpec/InvalidFeatureCategory and RSpec/MissingFeatureCategory into
RSpec/FeatureCategory.

This not only reduces the amount of code but also side-steps potential
confusion with RuboCop rule Gemfile/MissingFeatureCategory.

* Allow FeatureCategories#check to accept nil value nodes
* Make FeatureCategories#suggestion_message private again

* Disable Lint/RedundantRequireStatement for Ruby 3.2
* Skip short-hand syntax for Style/HashSyntax for Ruby 3.2

Add `namespaces` to the list of forbidden tables, in order to recude the
risk of LWLock saturation.

See https://gitlab.com/groups/gitlab-org/-/epics/11543.

Signed-off-by: Rémy Coutable <remy@rymai.me>

- prefer data-driven approaches instead
  https://gitlab.com/groups/gitlab-org/-/epics/7374

- allows to give more context to why something is for
  saas only.

When fabricate_via_api! is called on given resources,
raise an issue and recommend using the resource factory
Add new cop to grace period

- ee/lib/**
- rubocop/**
- scripts/**
- sidekiq_cluster/**
- tooling/**

We no longer support Ruby 2.7 and upgraded to Rails 7 lately.

With that, we can:
* Remove some Ruby specific RuboCop configurations
* Target Rails Version 7.0
* Disable and remove `Lint/LastKeywordArgument` cop rule
* Remove related CI job

- encourage use of helper

This fixes an error that appears when a migration contains a nested
class that does not have a parent class.

Stock ActiveRecord STI works by storing the class name in the database,
which is heavy and unnecessary. EnumInheritance introduces a way to use
STI through model defined enums.

Changelog: other

EE: true
Changelog: changed

Auto-corrected via:
  rubocop -a --only Style/RedundantFreeze scripts spec tooling qa

Some places are calling strong_memoize_with() without parameters. They
should be using strong_memoize_attr() instead. Update the RuboCop to
reflect this.

The `Graphql/GIDExpectedType` cop enforces that we pass an
`expected_type:` param when calling `Gitlab.object_from_id`, but our
Global IDs validate that the object resolved from a Global ID matches
the model which does the same check as the :expected_type param.

- https://gitlab.com/gitlab-org/gitlab/-/blob/45b3c596ef8b181bc893bd3b71613edf66064936/app/graphql/types/global_id_type.rb#L98
- https://gitlab.com/gitlab-org/gitlab/-/blob/ac2116a0abbf98019145cbffa0fbe569c7e2f978/app/graphql/gitlab_schema.rb#L128

https://gitlab.com/gitlab-org/gitlab/-/issues/417093

Fixes some `Naming/InclusiveLanguage` offenses.

This commit:
- Moves some of the settings Service Ping metrics to instrumentation
  classes.
- Adds PrometheusMetric to handle metrics querying Prometheus (being
  wrapped by the #with_prometheus call) without resorting to evaluate
  the metric value block in another context.
- Fixes some specs that were broken by migrating the metrics to the
  instrumentation class framework.