Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cannot switch to latest coding standard #414

Open
unicoder88 opened this issue Jul 10, 2022 · 5 comments
Open

Cannot switch to latest coding standard #414

unicoder88 opened this issue Jul 10, 2022 · 5 comments
Labels

Comments

@unicoder88
Copy link

Hi!

On Magento 2.4.3, I'm looking to upgrade to latest magento-coding-standard (v25), however, there's a conflict with webonyx/graphql-php.

    "require": {
        "php": ">=7.3",
        "webonyx/graphql-php": "^14.9",

My Magento version is stuck with v0.13.9, so I'm effectively unable to check against up-to-date code styles and stuck with standards v6.

I believe that coding standard should not technically depend on that, and we can safely remove any Magento code version requirements.

Agree?

@unicoder88 unicoder88 added the bug Something isn't working label Jul 10, 2022
@m2-assistant
Copy link

m2-assistant bot commented Jul 10, 2022

Hi @unicoder88. Thank you for your report.
To speed up processing of this issue, make sure that you provided sufficient information.

Add a comment to assign the issue: @magento I am working on this


@gowrizrh
Copy link

gowrizrh commented Aug 8, 2022

Having the same issue on version 2.4.3-p2

@guvra
Copy link

guvra commented Aug 9, 2022

I think that the issue is that the require section of this standard contains a lot of things that should be in require-dev instead.

The standard itself should only require two things:

  • Any version of codesniffer that is compatible with it (the requirement must be as loose as possible)
  • Compatible versions of PHP

@fredden
Copy link
Member

fredden commented Aug 10, 2022

I think that the issue is that the require section of this standard contains a lot of things that should be in require-dev instead.

I've checked just now and all the packages listed in require are used in the standards, not just its tests.

@fredden
Copy link
Member

fredden commented Aug 10, 2022

It looks like this is fixed since Magento version 2.4.4.

https://github.com/magento/magento2/blob/2.4.3/composer.json#L82 - "webonyx/graphql-php": "^0.13.8",
https://github.com/magento/magento2/blob/2.4.4/composer.json#L86 - "webonyx/graphql-php": "~14.11.3",
https://github.com/magento/magento2/blob/2.4.5/composer.json#L81 - "webonyx/graphql-php": "~14.11.6",

https://github.com/magento/magento-coding-standard/blob/v25/composer.json#L12 - "webonyx/graphql-php": "^14.9",

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants