-
Notifications
You must be signed in to change notification settings - Fork 57
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
Tools - fix loading versions installed by composer 2.0, upgrade tools versions and config #218
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
https://ci.appveyor.com/project/zdenekdrahos/phpqa/builds/37565960#L207 Version "not installed" printed by `phpqa tools` https://getcomposer.org/upgrade/UPGRADE-2.0.md vendor/composer/installed.json format changed: packages are now wrapped into a "packages" top level key instead of the whole file being the package array
https://github.com/EdgedesignCZ/phpqa/pull/218/checks?check_run_id=1813978113#step:5:127 Package jakub-onderka/php-parallel-lint is abandoned, you should avoid using it. Use php-parallel-lint/php-parallel-lint instead. Package jakub-onderka/php-console-color is abandoned, you should avoid using it. Use php-parallel-lint/php-console-color instead. Package jakub-onderka/php-console-highlighter is abandoned, you should avoid using it. Use php-parallel-lint/php-console-highlighter instead.
composer outdated -D composer config platform.php 5.4 composer require phpmd/phpmd:* pdepend/pdepend:* composer config platform --unset composer update nothing
https://github.com/EdgedesignCZ/phpqa/pull/218/checks?check_run_id=1814348576#step:5:210 Installation request for phpunit/phpunit ~7.0... https://github.com/EdgedesignCZ/phpqa/pull/218/checks?check_run_id=1814385995 PHP Fatal error: Declaration of Edge\QA\OptionsTest::setUp() must be compatible with PHPUnit\Framework\TestCase::setUp(): void in OptionsTest.php on line 25 https://github.com/EdgedesignCZ/phpqa/pull/218/checks?check_run_id=1814481036#step:6:48 Unknown option "--progress" https://github.com/sebastianbergmann/phpcpd/blob/master/ChangeLog.md 6.0.0 - 2020-08-13 Removed: The --names CLI option has been removed; use the --suffix CLI option instead The --progress CLI option has been removed
zdenekdrahos
force-pushed
the
tools-2021
branch
5 times, most recently
from
February 2, 2021 12:49
f7c2998
to
0c89b2e
Compare
This was referenced Feb 2, 2021
Closed
zdenekdrahos
force-pushed
the
tools-2021
branch
5 times, most recently
from
February 2, 2021 13:32
dccad2d
to
fe0537b
Compare
zdenekdrahos
force-pushed
the
tools-2021
branch
2 times, most recently
from
February 2, 2021 14:02
5ba50dc
to
593e90a
Compare
Closed
zdenekdrahos
force-pushed
the
tools-2021
branch
3 times, most recently
from
February 3, 2021 09:46
76b0b52
to
bf83e32
Compare
…obo2 https://github.com/EdgedesignCZ/phpqa/pull/218/checks?check_run_id=1814680025#step:6:16 Error: Class 'Twig_Loader_Filesystem' not found https://github.com/EdgedesignCZ/phpqa/pull/218/checks?check_run_id=1814733968#step:6:818 PHP Fatal error: Uncaught TypeError: Argument 1 passed to Symfony\Component\Process\Process::__construct() must be of the type array, string given
zdenekdrahos
force-pushed
the
tools-2021
branch
3 times, most recently
from
February 3, 2021 10:25
3445c90
to
5b847d0
Compare
https://github.com/EdgedesignCZ/phpqa/pull/218/checks?check_run_id=1821520385#step:7:28 Could not resolve config path to /home/runner/work/phpqa/phpqa/build//vendor https://github.com/vimeo/psalm/releases/tag/4.0.0 Change default on resolveFromConfigFile to "true". This means if your Psalm config is not in the same directory that you run Psalm from, and you want to keep all the paths correct, you'll need to add resolveFromConfigFile="false" to maintain existing path resolution https://github.com/EdgedesignCZ/phpqa/pull/218/checks?check_run_id=1821586719#step:7:28 Element '{https://getpsalm.org/schema/config}resolveFromConfigFile': This element is not expected. https://github.com/EdgedesignCZ/phpqa/pull/218/checks?check_run_id=1821638632#step:7:23 psalm v1: The attribute 'resolveFromConfigFile' is not allowed.
zdenekdrahos
force-pushed
the
tools-2021
branch
from
February 3, 2021 10:30
5b847d0
to
4df3a71
Compare
php-cs-fixer hotfix (6ef3519) is outdated New php versions are supported. It's necessary only for old installations with Symfony2: - friendsofphp/php-cs-fixer v2.18.2 requires symfony/filesystem ^3.0 || ^4.0 || ^5.0 - vimeo/psalm 4.4.0 requires symfony/console ^3.4.17 || ^4.1.6 || ^5.0 - vimeo/psalm 3.0.0 requires symfony/console ^3.0||^4.0
zdenekdrahos
changed the title
Tools - fix loading versions installed by composer 2.0, upgrade tools
Tools - fix loading versions installed by composer 2.0, upgrade tools versions and config
Feb 4, 2021
2 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
composer outdated -D
phpstan xml error -simplexml_load_file(): build//phpstan.xml:1: parser error : Start tag expected, '<' not found
- phpstan xml error + phpmetrics v1 =Hoa main file (Core.php) must be included once.
Could not resolve config path to /home/runner/work/phpqa/phpqa/build//vendor
, same error for php 7.4, psalm 1, php 7.1 works - change in version? related to Psalm: check if directory exists before include them at <ignoreFiles> #206? No, default resolveFromConfigFile changed in v4