Skip to content
Snippets Groups Projects

Add frozen_string_literal to spec/features

Merged Thong Kuah requested to merge frozen_string_spec_features into master
All threads resolved!

What does this MR do?

Add frozen_string_literal to spec/features

https://gitlab.com/gitlab-org/gitlab-ce/issues/59758

Does this MR meet the acceptance criteria?

Conformity

Performance 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 🤖 GitLab Bot 🤖

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Thong Kuah resolved all threads

    resolved all threads

  • Thong Kuah added 87 commits

    added 87 commits

    Compare with previous version

  • unassigned @tkuah

  • Tiger Watson approved this merge request

    approved this merge request

  • Author Maintainer

    @rymai Can you please review this MR ? Thanks

  • assigned to @rymai

  • Thong Kuah resolved all threads

    resolved all threads

  • @tkuah Thank you, looks good to me! :heart: :yellow_heart: :green_heart: :rocket:

    Do you have an EE MR for EE-specific spec files too?

  • Rémy Coutable approved this merge request

    approved this merge request

  • Rémy Coutable mentioned in commit aa0d8b4c

    mentioned in commit aa0d8b4c

  • Author Maintainer

    Thanks @rymai !

    Do you have an EE MR for EE-specific spec files too?

    Not yet, that will come soon when I have some downtime :)

  • 🤖 GitLab Bot 🤖 changed the description

    changed the description

  • Please register or sign in to reply
    Loading