-
Notifications
You must be signed in to change notification settings - Fork 4
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
OEL-2794: Update components to run on Drupal 10.2.x #252
Conversation
e6f4357
to
ea540ba
Compare
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.
we can move on to php 8.2 in docker compose for local development
README.md
Outdated
@@ -28,6 +28,15 @@ Ideally this project should be managed with [Docker](https://www.docker.com/get- | |||
|
|||
Check the [composer.json](composer.json) for required PHP version and other dependencies. | |||
|
|||
#### Drupal 10.1 |
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.
remove this as oe_content is a dev dependency
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.
Removed
composer.json
Outdated
@@ -103,7 +103,19 @@ | |||
}, | |||
"_readme": [ | |||
"Explicit requirement of nikic/php-parser ^4 as later versions are not compatible with grumphp @see https://github.com/phpro/grumphp/issues/1119" | |||
] | |||
], | |||
"patches": { |
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.
do we really need this in our project?
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.
Removed
No description provided.