Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 16, 2025

Upgrades elasticsearch client dependency constraints to support v9.x while dropping v7.x support.

Changes

  • composer.json (require-dev): ^7.0 || ^8.0^8.0 || ^9.0
  • src/elasticsearch/composer.json (require): ^8.0^8.0 || ^9.0
  • src/sentry/composer.json (suggest): ^7.0|^8.0^8.0 || ^9.0
  • src/telescope-elasticsearch/composer.json (suggest): ^7.17.0||^8.8.0^8.0 || ^9.0
  • src/sentry/src/Tracing/Aspect/ElasticsearchAspect.php: Removed v7.x class references (Elasticsearch\Client::*), keeping only v8.x+ references (Elastic\Elasticsearch\Traits\ClientEndpointsTrait::*)
  • .github/workflows/test-components.yml: Added elasticsearch dependency testing job that validates compatibility with ^8.0 and ^9.0 versions across PHP 8.2, 8.3, and 8.4

All composer.json files validated successfully.

Original prompt

升级依赖包(包括主包、子包)
原:"elasticsearch/elasticsearch": "^7.0 || ^8.0"
升级:"elasticsearch/elasticsearch": "^8.0 || ^9.0"


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…0||^9.0

Co-authored-by: huangdijia <8337659+huangdijia@users.noreply.github.com>
Copilot AI changed the title [WIP] Update elasticsearch dependencies to version 8.0 and 9.0 Upgrade elasticsearch/elasticsearch dependency from ^7.0||^8.0 to ^8.0||^9.0 Nov 16, 2025
Copilot AI requested a review from huangdijia November 16, 2025 06:16
@huangdijia
Copy link
Contributor

@copilot src/sentry/src/Tracing/Aspect/ElasticsearchAspect.php remove 7.x support

Co-authored-by: huangdijia <8337659+huangdijia@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Nov 16, 2025

@copilot src/sentry/src/Tracing/Aspect/ElasticsearchAspect.php remove 7.x support

Removed v7.x class references from ElasticsearchAspect.php (commit a32368c)

@huangdijia
Copy link
Contributor

@copilot .github/workflows/test-components.yml 增加elasticsearch依赖包测试

Co-authored-by: huangdijia <8337659+huangdijia@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Nov 16, 2025

@copilot .github/workflows/test-components.yml 增加elasticsearch依赖包测试

Added elasticsearch testing job to test-components.yml workflow with PHP 8.2-8.4 and elasticsearch ^8.0 and ^9.0 versions (commit 95499ec)

@friendsofhyperf friendsofhyperf deleted a comment from coderabbitai bot Nov 16, 2025
@coderabbitai
Copy link

coderabbitai bot commented Nov 16, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Comment @coderabbitai help to get the list of available commands and usage tips.

@huangdijia huangdijia marked this pull request as ready for review November 16, 2025 08:19
@huangdijia huangdijia merged commit 5f6af44 into 3.2 Nov 16, 2025
76 checks passed
@huangdijia huangdijia deleted the copilot/upgrade-elasticsearch-dependencies branch November 16, 2025 08:20
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.

2 participants