Skip to content

Remove unnecessary default for required properties

Tomas Vik requested to merge 324169-remove-unnecessary-default into master

What does this MR do?

This is a follow up to this review comment: !67495 (comment 652081376)

@afontaine noticed that some of the MR widget Vue components use both require: true and default: () => ({}). The default is unnecessary for required properties.

There are also existing components in the same folder that are already correct:

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

Does this MR contain changes to processing or storing of credentials or tokens, authorization and authentication methods or other items described in the security review guidelines? If not, then delete this Security section.

  • 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

Related to #324169 (closed)

Edited by Tomas Vik

Merge request reports

Loading