Skip to content

Conversation

@oliverklee
Copy link
Collaborator

No description provided.

@coveralls
Copy link

coveralls commented Feb 10, 2025

Coverage Status

coverage: 49.156%. remained the same
when pulling 6b4ec7c on bugfix/import-throws
into 5c440f3 on main.

Copy link
Collaborator

@JakeQZ JakeQZ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Impressed that Stan managed to identify the FQN of the exception class without the use statement.

@JakeQZ JakeQZ merged commit ceec94e into main Feb 10, 2025
21 checks passed
@JakeQZ JakeQZ deleted the bugfix/import-throws branch February 10, 2025 10:40
@oliverklee
Copy link
Collaborator Author

Impressed that Stan managed to identify the FQN of the exception class without the use statement.

It didn't - that's why this change fixes some PHPStan warnings. ;-)

oliverklee added a commit that referenced this pull request Feb 10, 2025
@JakeQZ
Copy link
Collaborator

JakeQZ commented Feb 11, 2025

Impressed that Stan managed to identify the FQN of the exception class without the use statement.

It didn't - that's why this change fixes some PHPStan warnings. ;-)

You're right, it just assumed FQN from the current namespace.

JakeQZ pushed a commit that referenced this pull request Feb 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants