Skip to content

Conversation

@jigar-shah-acquia
Copy link
Contributor

Motivation
Fixes #CLI-1679

Proposed changes

  • Revert the use of custom forks and dev branches introduced for PHP 8.5 compatibility.

Update composer.json to use the latest stable upstream releases for:

  • amplitude-php (>= 1.0.9)

  • amphp/parallel (>= 2.3.3)

  • revolt/event-loop (>= 1.0.8)
    Remove references to custom repositories and branch aliases.

  • Ensure Acquia CLI continues to function correctly on PHP 8.5 using officially supported dependencies.

Alternatives considered
N/A

Testing steps

  1. Follow the contribution guide to set up your development environment or download a pre-built acli.phar for this PR.
  2. If running from source, clear the kernel cache to pick up new and changed commands: ./bin/acli ckc
  3. Check for regressions: (add specific steps for this pr)
  4. Check new functionality: (add specific steps for this pr)

Copilot AI review requested due to automatic review settings December 29, 2025 06:25
@jigar-shah-acquia jigar-shah-acquia self-assigned this Dec 29, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR aims to revert the use of custom forks and development branches that were previously introduced for PHP 8.5 compatibility. The changes update the zumba/amplitude-php package constraint from ^1.0.4 to ^1.0.9 to use the latest stable upstream release instead of a custom fork.

Key changes:

  • Update zumba/amplitude-php version constraint from ^1.0.4 to ^1.0.9

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant