Skip to content

Simplify installation instructions#436

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

Open
wants to merge 1 commit into
base:develop
Choose a base branch
from

Conversation

fredden
Copy link
Member

@freddenfredden 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

@Den4ikDen4ik 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

Den4ik
Den4ik previously approved these changes Feb 6, 2023
Copy link

@Den4ikDen4ik 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
@freddenfredden dismissed stale reviews from Den4ik and ihor-sviziev via c534321April 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
@freddenfreddenforce-pushed the simplify-install-instructions branch 2 times, most recently from 6d8a941 to afe826dCompareSeptember 20, 2023 15:11
@freddenfreddenforce-pushed the simplify-install-instructions branch from afe826d to 2d3d36eCompareDecember 8, 2023 16:37
ihor-sviziev
ihor-sviziev previously approved these changes Dec 11, 2023
ihor-sviziev
ihor-sviziev previously approved these changes Oct 9, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment
close