Skip to content

Enhancement: Allow fetching constant name from exception - #570

Merged
Ocramius merged 1 commit into
Roave:masterfrom
localheinz:feature/constant
May 6, 2020
Merged

Ocramius merged 1 commit into
Roave:masterfrom
localheinz:feature/constant

Conversation

@localheinz

@localheinz localheinz commented May 6, 2020 •

Copy link
Copy Markdown
Contributor

This PR

  • allows fetching the constant name from an UnableToCompileNode exception

Fixes #569.

Comment thread src/NodeCompiler/Exception/UnableToCompileNode.php Outdated
@Ocramius Ocramius self-assigned this May 6, 2020
@Ocramius Ocramius modified the milestones: 3.5.1, 4.1.0 May 6, 2020

@Ocramius Ocramius left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM 👍

Comment thread src/NodeCompiler/Exception/UnableToCompileNode.php Outdated

@Ocramius Ocramius left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Accessor approach is indeed good enough for our purposes 👍

@Ocramius
Ocramius merged commit cf9ff24 into Roave:master May 6, 2020
@Ocramius

Ocramius commented May 6, 2020

Copy link
Copy Markdown
Member

Thanks @localheinz!

@localheinz
localheinz deleted the feature/constant branch May 6, 2020 17:55
@localheinz

Copy link
Copy Markdown
Contributor Author

Thank you, @Ocramius!

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.

Add accessors to UnableToCompileNode exception

2 participants