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

Simplify installation instructions #436

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

fredden
Copy link
Member

@fredden fredden commented Feb 1, 2023

The installation instructions for this coding standard are unnecessarily complex. Composer plugins are capable of performing the necessary registration automatically, without requiring the user to manually set up (or alter) any scripts sections in composer.json. The main Magento repository uses dealerdirect/phpcodesniffer-composer-installer to register the coding standard; let's use the same here.

See https://github.com/magento/magento2/blob/11846a1a10539470f2fe1522030ff42d62daa562/composer.json#L110 / magento/magento2#36791

Fixes #390

Copy link

@Den4ik Den4ik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @fredden Thanks for your update.
Please review my comment

composer.json Show resolved Hide resolved
Den4ik
Den4ik previously approved these changes Feb 6, 2023
Copy link

@Den4ik Den4ik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Looks good for me

ihor-sviziev
ihor-sviziev previously approved these changes Apr 12, 2023
Den4ik
Den4ik previously approved these changes Apr 12, 2023
@fredden fredden dismissed stale reviews from Den4ik and ihor-sviziev via c534321 April 28, 2023 12:45
@sinhaparul
Copy link
Contributor

@magento import PR to magento-commerce/magento-coding-standard repository

@m2-github-services
Copy link
Contributor

@sinhaparul the Pull Request is successfully imported.

ihor-sviziev
ihor-sviziev previously approved these changes Jun 12, 2023
ihor-sviziev
ihor-sviziev previously approved these changes Aug 14, 2023
@fredden fredden force-pushed the simplify-install-instructions branch 2 times, most recently from 6d8a941 to afe826d Compare September 20, 2023 15:11
@fredden fredden force-pushed the simplify-install-instructions branch from afe826d to 2d3d36e Compare December 8, 2023 16:37
ihor-sviziev
ihor-sviziev previously approved these changes Dec 11, 2023
@fredden fredden force-pushed the simplify-install-instructions branch from 556edd4 to 7143ee5 Compare September 10, 2024 19:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Review in Progress
Development

Successfully merging this pull request may close these issues.

PHPStorm unable to run phpcs inspection
5 participants