-
Notifications
You must be signed in to change notification settings - Fork 1
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
Feature/SRM-871 d10 update #89
Conversation
This reverts commit 13c391b. Reverted token conditions changes.
Merged develop changes.
2186d40
to
f330a83
Compare
Merge develop branch changes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, couple of comments however
"squizlabs/php_codesniffer": "3.7.1", | ||
"dealerdirect/phpcodesniffer-composer-installer": "^0.7", | ||
"drupal/coder": "^8.3.10", | ||
"drupal/console": "^1.0", | ||
"drupal/drupal-extension": "^4.1", | ||
"drupal/console": "dev-feature/Symfony-config-d10-compatibility", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a release now that supports this? I expect since D10 isn't as new now, there may well be
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@krakerag there is still a open PR for this - hechoendrupal/drupal-console#4348
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm sad that they haven't merged it yet, I guess that will have to do. As long as we lock a working version in the lock file we should be fine.
composer.json
Outdated
@@ -4,7 +4,7 @@ | |||
"license": "GPL-2.0-or-later", | |||
"type": "drupal-module", | |||
"require": { | |||
"dpc-sdp/tide_core": "^3.1.0" | |||
"dpc-sdp/tide_core": "dev-feature/SRM-870-d10-update" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just remember to clean this up before merge (or after merge in a new PR etc)
No description provided.