Skip to content

Fix Graphql/Descriptions FOSS resolver offenses [RUN AS-IF-FOSS]

What does this MR do?

!46721 (merged) introduced a new rubocop that enforced periods at the end of GraphQL description strings. Existing offenses were added to .rubocop_manual_todo.yml.

This change fixes offenses only for files in app/graphql/resolvers, and subdirectories, and removes the exclusion of them from .rubocop_manual_todo.yml.

The changes were generated using autocorrect:

    bundle exec rubocop app/graphql/resolvers \
      --auto-correct \
      --only Graphql/Descriptions

Reviewing tips

  1. Single-file reviewing might be useful (on the Changes tab, click the cog icon, then check Show one file at a time).
  2. The total number of changes lines in this MR is very high, but this is mainly due to the changes under docs/ which are auto-generated (and verified during CI).

Does this MR meet the acceptance criteria?

Conformity

Related to #292426 (closed)

Edited by Luke Duncalfe

Merge request reports

Loading