Skip to content

Subscriptions: Fix reserved keyword variable names phpcs reports #8540

New issue
New issue
Open
Open
Subscriptions: Fix reserved keyword variable names phpcs reports#8540

Description

@reykjalin

reykjalin
openedon Apr 1, 2024

Part of #8438.

Description

Remove all the Universal.NamingConventions.NoReservedKeywordParameterNames excludes from phpcs.xml.dist and fix the issues reported when running npm run lint:php in the following files:

  • tests/unit/helpers/class-wc-helper-subscription.php
  • tests/unit/helpers/class-wc-helper-subscriptions.php
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Assignees

No one assigned

    Labels

    focus: architectureProduct architecture and engineering best practicesgood first issueThe issue is a good candidate for the first community contribution/for a newcomer to the team.priority: lowThe issue/PR is low priority—not many people are affected or there’s a workaround, etc.type: technical debtThis issue/PR represents/solves the technical debt of the project.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions