Skip to content

feat: produce deprecation when use activity method without the activity attribute#677

Merged
xepozz merged 8 commits intomasterfrom
activity-method
Feb 23, 2026
Merged

feat: produce deprecation when use activity method without the activity attribute#677
xepozz merged 8 commits intomasterfrom
activity-method

Conversation

@xepozz
Copy link
Collaborator

@xepozz xepozz commented Dec 20, 2025

…ty attribute

What was changed

calling activity without an explicit attribute will produce E_USER_DEPRECATION warning

Why?

Checklist

  1. Closes [Bug] Methods without ActivityMethod attribute are registered as Activities (even magic) #382

  2. How was this tested:

  1. Any docs updates needed?

@xepozz xepozz requested a review from wolfy-j as a code owner December 20, 2025 08:07
@vercel
Copy link

vercel bot commented Dec 20, 2025

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

Project Deployment Actions Updated (UTC)
php Ready Ready Preview, Comment Feb 22, 2026 1:54pm

Request Review

@xepozz
Copy link
Collaborator Author

xepozz commented Dec 20, 2025

should it be fully disabled in 3.0?

@xepozz xepozz merged commit 1ec2e61 into master Feb 23, 2026
214 of 216 checks passed
@xepozz xepozz deleted the activity-method branch February 23, 2026 04:16
xepozz added a commit that referenced this pull request Feb 25, 2026
…ty attribute (#677)

* feat: produce deprecation when use activity method without the activity attribute

* feat: add feature flag

* fix: make psalm pass

* fix: update baseline
xepozz added a commit that referenced this pull request Mar 3, 2026
* feat: produce deprecation when use activity method without the activity attribute (#677)

* feat: produce deprecation when use activity method without the activity attribute

* feat: add feature flag

* fix: make psalm pass

* fix: update baseline

* feat: Make search attribute type more lenient when parsing
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.

[Bug] Methods without ActivityMethod attribute are registered as Activities (even magic)

2 participants