Skip to content

Conversation

@NickDickinsonWilde
Copy link

Description

No issue... maybe I should've written an issue before a PR? (but .github/CONTRIBUTING.md is empty so unsure).

Upgrading a pile of Drupal sites to PHP 8.5. Almost all of them need the same 4-8 patches. Most of them still have some PHP 8.4 related patches. Having to merge JSON manually for each site is more work than I think needed. Instead, I want to have multiple patches files...

patches.json (or whatever name) with site specific patches
php84.patches.json with PHP 8.4 related patches for components that most of my sites have
php85.patches.json with PHP 8.5 related patches for common components.

Because packages in the patch file that don't exist on the project are silently ignored that is really convenient and reduces work on similar projects.

Example output:
image

Related tasks

Everything here needs to be done if the idea is approved - happy to do so, but wanted to get a POC out for my own personal use/testing

  • Documentation has been updated if applicable
  • Tests have been added
  • Does not break backwards compatibility OR a BC break has been discussed in the related issue(s). -- in current patch I am messaging that current patches-file config key is deprecated. But that should probably be discussed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant