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.

Sign up for GitHub

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

Jump to bottom

classExists does not have @psalm-assert class-string since 1.8.0 #185

Closed
vudaltsov opened this issue Apr 18, 2020 · 2 comments · Fixed by #186
Closed

classExists does not have @psalm-assert class-string since 1.8.0 #185

vudaltsov opened this issue Apr 18, 2020 · 2 comments · Fixed by #186

Comments

@vudaltsov
Copy link
Contributor

For some reason it was removed in efafa74#diff-793d5a51af95587404301a8a49a3a364L1506

@Ocramius , was it on purpose?
@Ocramius
Copy link
Contributor

probably not: can you send a patch with a test please?

@vudaltsov
Copy link
Contributor Author

Sure!

vudaltsov added a commit to vudaltsov/assert that referenced this issue Apr 18, 2020
@vudaltsov
Add @psalm-assert and @psalm-pure to Assert::classExists
9a2b484 
Closes webmozarts#185
@vudaltsov vudaltsov mentioned this issue Apr 18, 2020
Merged
BackEndTea pushed a commit that referenced this issue Apr 19, 2020
@vudaltsov
Add @psalm-assert and @psalm-pure to Assert::classExists (#186)
e692710 
Closes #185
@vudaltsov vudaltsov mentioned this issue Apr 19, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add @psalm-assert and @psalm-pure to Assert::classExists vudaltsov/assert
2 participants
@Ocramius @vudaltsov