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

PHPCS Integration Composer Package #91

Merged
merged 6 commits into from
Jan 29, 2024

Conversation

ObliviousHarmony
Copy link
Owner

All Submissions:

  • Have you checked for duplicate PRs?
  • Have you added an entry to the CHANGELOG.md file's [Unreleased] section?

Changes proposed in this Pull Request:

This pull request sets up a Composer package for the integration files and adds a new configuration option to make it possible to use it instead of the integrated reports. This is useful for cases where the path to the report files from the extension are not available. This is primarily the case when using a Docker environment but not using a remote extension.

How to test the changes in this Pull Request:

  1. Enable the phpCodeSniffer.autoloadPHPCSIntegration setting. It should give an error about a missing report.
  2. Add a path repository to a composer.json file in a project you want to test against.
  3. composer require -D "obliviousharmony/vscode-phpcs-integration"="*"
  4. Make sure that the extension now works
  5. Disable the setting and confirm it works.
  6. Remove the package and confirm it works.

This makes the appropriate configuration changes to
support distributing the package via
Packagist & Composer.
A new `phpCodeSniffer.autoloadPHPCSIntegration`
option allows for using an autoloaded set of
integration files from a Composer package
instead of a file path to them.
@ObliviousHarmony ObliviousHarmony merged commit 2122d4b into main Jan 29, 2024
2 checks passed
@ObliviousHarmony ObliviousHarmony deleted the add/composer-package-support branch January 29, 2024 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant