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

Add missing reset of context inside_general_use #11157

Merged
merged 1 commit into from
Feb 16, 2025

Conversation

MoonE
Copy link
Contributor

@MoonE MoonE commented Nov 12, 2024

Everywhere else this property is reset before returning, just here it seems to be missing.
Not sure what the ramifications are as the tests seem to pass either way.

See #7195

@MoonE MoonE changed the base branch from 5.x to 6.x January 26, 2025 22:35
@MoonE MoonE force-pushed the missing-reset-context-inside-general-use branch from 3a5063c to f4e4dc4 Compare January 26, 2025 22:36
@orklah orklah added the release:fix The PR will be included in 'Fixes' section of the release notes label Feb 16, 2025
@orklah
Copy link
Collaborator

orklah commented Feb 16, 2025

Thanks!

@orklah orklah merged commit e3fe092 into vimeo:6.x Feb 16, 2025
51 of 52 checks passed
@MoonE MoonE deleted the missing-reset-context-inside-general-use branch February 16, 2025 18:41
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.

2 participants