Skip to content

Do not commit the generated schema [RUN AS-IF-FOSS]

Alex Kalderimis requested to merge ajk-246317-no-generated-schema into master

What does this MR do?

Addresses #246317 (closed) by removing the compiled schema artifacts from the repo, and no longer validating their inclusion.

The rationale for this is that these files are not used by us, and are not necessary for clients. The ones I use regularly (GraphIQL, insomnia and graphqurl) all use introspection queries, which have a number of advantages (not least always being accurate representations of the schema).

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Alex Kalderimis

Merge request reports

Loading