Skip to content

Fix #628 by enabling opt-in to nullable scopes#644

Open
grantholle wants to merge 4 commits intoJosephSilber:masterfrom
grantholle:loose-scoping
Open

Fix #628 by enabling opt-in to nullable scopes#644
grantholle wants to merge 4 commits intoJosephSilber:masterfrom
grantholle:loose-scoping

Conversation

@grantholle
Copy link
Contributor

This tries to rectify the aftermath of #608 by allowing you to opt-in to allowing scope to be null.

To enable pre 1.0.1 functionality, call allowNullScope() when setting the scope.

Bouncer::scope()
    ->to(1)
    ->allowNullScope();

@grantholle
Copy link
Contributor Author

I don't know if I solved the failing test the right way?

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.

1 participant