Skip to content

Snippets

Why

Several files in the snippets domain have entries in the rubocop todo's for bounded contexts.

For example

- 'app/models/snippet.rb'
- 'ee/app/models/ee/snippet.rb'
- 'app/graphql/types/permission_types/snippet.rb'
- 'app/models/personal_snippet.rb'
- 'app/models/project_snippet.rb'

There may be other related entries.

There are several of these overlapping in https://gitlab.com/gitlab-org/gitlab/-/blob/master/.rubocop_todo/gitlab/namespaced_class.yml . I'm not 100% certain but fixing the bounded context entries may also address the namespaced class entries as well.

What

These need to be added to the Snippets bounded context namespace. So in other words, Snippet needs to become Snippets::Snippet to comply with bounded contexts. We need to do this for all Snippet related code that is not currently in the Snippets bounded context.

Edited by Hunter Stewart