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

dev/drupal#176 - allow symfony 6 #25499

Merged
merged 1 commit into from Feb 2, 2023

Conversation

demeritcowboy
Copy link
Contributor

Overview

https://lab.civicrm.org/dev/drupal/-/issues/176

Before

Have to use a modified version of core to install on drupal 10

After

Can install on drupal 10 (along with #25054)

Technical Details

Note composer.lock is not changed here, so other cmses will still get what they are currently getting.

Comments

This and the other PR are the last remaining "hacks" needed.

@civibot
Copy link

civibot bot commented Feb 2, 2023

No issue was found matching the number given in the pull request title. Please check the issue number.

@civibot
Copy link

civibot bot commented Feb 2, 2023

(Standard links)

@civibot civibot bot added the master label Feb 2, 2023
@demeritcowboy
Copy link
Contributor Author

jenkins retest this please

@seamuslee001
Copy link
Contributor

@demeritcowboy can you please rebase this and run composer update php and commit the change to the lockfile (should likely only be a hash) as I see in the console Warning: The lock file is not up to date with the latest changes in composer.json. You may be getting outdated dependencies. It is recommended that you run composer updateorcomposer update .

@demeritcowboy
Copy link
Contributor Author

Thanks yes good point. Have updated.

@seamuslee001
Copy link
Contributor

Jenkins re test this please

@demeritcowboy
Copy link
Contributor Author

Woo!

@demeritcowboy demeritcowboy merged commit 4b4fe29 into civicrm:master Feb 2, 2023
@demeritcowboy demeritcowboy deleted the symfony-composer-json branch February 2, 2023 23:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants