Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow concatenation between (string|int) and similar types #11364

Merged
merged 3 commits into from
Mar 20, 2025

Conversation

haas-dtv
Copy link
Contributor

Fixes #11337

@danog
Copy link
Collaborator

danog commented Mar 20, 2025

Looks okay, can you also add a few unit tests please?

@danog danog added the release:fix The PR will be included in 'Fixes' section of the release notes label Mar 20, 2025
@haas-dtv
Copy link
Contributor Author

I've added a few tests

@danog danog merged commit 514e954 into vimeo:master Mar 20, 2025
50 of 51 checks passed
@danog
Copy link
Collaborator

danog commented Mar 20, 2025

Btw, could you also submit this against 6.x

@haas-dtv haas-dtv deleted the fix/11337 branch March 20, 2025 13:02
danog added a commit that referenced this pull request Mar 20, 2025
Backport #11364 to 6.x
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:fix The PR will be included in 'Fixes' section of the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants