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

DT-1356: Bump drupal-check version #4072

Closed
danepowell opened this issue Mar 23, 2020 · 4 comments · Fixed by #4076
Closed

DT-1356: Bump drupal-check version #4072

danepowell opened this issue Mar 23, 2020 · 4 comments · Fixed by #4076
Labels
Enhancement A feature or feature request

Comments

@danepowell
Copy link
Contributor

danepowell commented Mar 23, 2020

https://github.com/mglaman/drupal-check/releases/tag/1.1.1

Need to test this extensively, especially on Pipelines, given all of the previous issues with this package.

@aweingarten
Copy link
Contributor

aweingarten commented Mar 25, 2020

Just want to bump the priority of this given that D9 is officially coming out in early June. drupal-check/upgrade_status is the recommended tool from Acquia for D9 readiness and has even been funded by Acquia. Getting BLT to be compatible with the recommended Acquia tool for D9 is kind of important. Right now there seem to composer version constraints from BLT preventing its installation.

http://hojtsy.hu/blog/2019-apr-29/estimate-your-sites-drupal-9-compatibility-easily-upgrade-status
https://twitter.com/Dries/status/1136968043979005953
https://dev.acquia.com/drupal9

@mikemadison13
Copy link
Contributor

It looks like trying to add drupal/upgrade_status to a BLT 11.x project hits a lot of composer issues right now. Even though Upgrade Status itself doesn't require drupal-check, they do cross-contaminate a lot of the underlying phpstan dependencies. Obviously we need to make sure that BLT continues to be stable (so i dunno that we want to go back to ^ definitions for Drupal Check) but I do agree if we can unblock upgrade status ASAP that would be ideal!

composer require drupal/upgrade_status
Using version ^2.0 for drupal/upgrade_status
./composer.json has been updated
Gathering patches for root package.
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Conclusion: don't install drupal/upgrade_status 2.0.0
    - Conclusion: don't install drupal/upgrade_status 2.0.0-beta1
    - Conclusion: don't install drupal/upgrade_status 2.0.0-alpha1
    - Conclusion: remove mglaman/phpstan-drupal 0.11.14
    - Installation request for drupal/upgrade_status ^2.0 -> satisfiable by drupal/upgrade_status[2.x-dev, 2.0.0, 2.0.0-beta1, 2.0.0-alpha1].
    - Conclusion: don't install mglaman/phpstan-drupal 0.11.14
    - drupal/upgrade_status 2.x-dev requires phpstan/phpstan-deprecation-rules ^0.12.0 -> satisfiable by phpstan/phpstan-deprecation-rules[0.12.0, 0.12.1, 0.12.2, 0.12.x-dev].
    - phpstan/phpstan-deprecation-rules 0.12.0 conflicts with mglaman/phpstan-drupal[0.11.14].
    - phpstan/phpstan-deprecation-rules 0.12.1 conflicts with mglaman/phpstan-drupal[0.11.14].
    - phpstan/phpstan-deprecation-rules 0.12.2 conflicts with mglaman/phpstan-drupal[0.11.14].
    - phpstan/phpstan-deprecation-rules 0.12.x-dev conflicts with mglaman/phpstan-drupal[0.11.14].
    - Installation request for mglaman/phpstan-drupal (locked at 0.11.14) -> satisfiable by mglaman/phpstan-drupal[0.11.14].
``

@danepowell
Copy link
Contributor Author

Have you tried using Composer version aliases? That's the normal workaround when you want to use a different version of a dependency pinned by an upstream package.

Also just for context here... every time that we bump the drupal-check version (or allow users to upgrade it), it breaks in some major and unforeseen way (even during "patch" upgrades). While I appreciate the priority of this issue, that's why we can't rush into supporting the new version or allow users to update unconstrained, until we are sure it is stable. See #3996

@danepowell danepowell changed the title Bump drupal-check version DT-1356: Bump drupal-check version Mar 25, 2020
@danepowell danepowell added this to the Backlog milestone Mar 25, 2020
@danepowell
Copy link
Contributor Author

Specifically, I'd recommend anyone thinking they are blocked by this to try something like:
composer require mglaman/drupal-check:"1.1.1 as 1.0.14" --no-update && composer update

@danepowell danepowell added the Enhancement A feature or feature request label Mar 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement A feature or feature request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants