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

adding composer code #1950

Merged
merged 5 commits into from
Sep 10, 2021
Merged

adding composer code #1950

merged 5 commits into from
Sep 10, 2021

Conversation

admiralAwkbar
Copy link
Collaborator

This closes #1920

This will add the loop to run a composer install if the variable VALIDATE_PHP_PSALM is set to true and we find PHP code to validate.

This is now added to a new loop to help with additional linters that may need some additional work before a run

@admiralAwkbar
Copy link
Collaborator Author

Had to add some additional logic to look for composer.json file in the users repository. If the file is found, it will go to the location and run composer install
This should now make sure all libs and such are loaded for proper Psalm validation

@admiralAwkbar admiralAwkbar merged commit a3cce70 into master Sep 10, 2021
@admiralAwkbar admiralAwkbar deleted the psamlInstall branch September 10, 2021 14:43
@admiralAwkbar admiralAwkbar restored the psamlInstall branch December 21, 2021 15:28
@admiralAwkbar admiralAwkbar deleted the psamlInstall branch December 21, 2021 15:31
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.

Psalm not running
1 participant