Skip to content

[WIP] Fix deprecation warning for easywechat-composer package#2976

Closed
overtrue with Copilot wants to merge 1 commit into
6.xfrom
copilot/fix-php8-4-deprecation-warning
Closed

[WIP] Fix deprecation warning for easywechat-composer package#2976
overtrue with Copilot wants to merge 1 commit into
6.xfrom
copilot/fix-php8-4-deprecation-warning

Conversation

Copilot AI commented Feb 25, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor
  • Analyze the issue: PHP 8.4 deprecated implicit nullable parameter pattern in easywechat-composer/easywechat-composer vendor package (used in 5.x branch)
  • Confirm 6.x codebase is already PHP 8.4 compatible (no deprecated patterns found)
  • Find PHP 8.4 deprecated patterns in 5.x branch: src/OpenPlatform/Application.php has multiple string $param = null and AccessToken $param = null deprecated patterns
  • Fix src/OpenPlatform/Application.php in 5.x: change string $refreshToken = null → ?string $refreshToken = null and AccessToken $accessToken = null → ?AccessToken $accessToken = null
Original prompt

This section details on the original issue you should resolve

<issue_title>5.x 版中依赖的另外一个包 easywechat-composer/easywechat-composer，在php8.4下警告</issue_title>
<issue_description>5.x 版中依赖的另外一个包 easywechat-composer/easywechat-composer，在php8.4下警告如下：

Deprecation Notice: EasyWeChatComposer\ManifestManager::__construct(): Implicitly marking parameter $manifestPath as nullable is deprecated, the explicit nullable type must be used instead in /home/koogua/phalcon-v5-china-project/ctc-pro/vendor/easywechat-composer/easywechat-composer/src/ManifestManager.php:42
```</issue_description>

<agent_instructions>PR to fix it.</agent_instructions>

## Comments on the Issue (you are @copilot in this section)

<comments>
<comment_new><author>@overtrue</author><body>
https://github.com/mingyoung/easywechat-composer 去这里反馈或者提 PR</body></comment_new>
</comments>

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@vercel

vercel Bot commented Feb 25, 2026 •
edited
Loading

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
easywechat Ready Ready Preview, Comment Feb 25, 2026 2:34pm

@overtrue overtrue marked this pull request as ready for review February 25, 2026 14:52
Copilot AI review requested due to automatic review settings February 25, 2026 14:52
Copilot AI reviewed Feb 25, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

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

@overtrue overtrue closed this Feb 25, 2026
Copilot AI requested a review from overtrue February 25, 2026 14:54
Copilot stopped work on behalf of overtrue due to an error February 25, 2026 14:54
The session was cancelled by the user.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments
@overtrue overtrue Awaiting requested review from overtrue

Assignees

@overtrue overtrue

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5.x 版中依赖的另外一个包 easywechat-composer/easywechat-composer，在php8.4下警告

3 participants

@overtrue