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

Wrap entrypoints into IIFE to protect their variables #5366

Merged
merged 3 commits into from
Mar 11, 2021

Conversation

weirdan
Copy link
Collaborator

@weirdan weirdan commented Mar 10, 2021

Fixes #5359

src/Psalm/IssueBuffer.php Show resolved Hide resolved
tests/EndToEnd/PsalmRunnerTrait.php Show resolved Hide resolved
src/psalm.php Show resolved Hide resolved
@weirdan weirdan requested a review from muglug March 10, 2021 23:16
@muglug muglug merged commit 205fdd1 into vimeo:master Mar 11, 2021
@muglug
Copy link
Collaborator

muglug commented Mar 11, 2021

Thanks, this never occurred to me as a potential problem, stupidly

This was referenced Mar 15, 2021
This was referenced Mar 17, 2021
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.

Uncaught RuntimeException after changing the $config global in an autoloader
2 participants