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

add location to DynamicProxy #6346

Open
sanderr opened this issue Aug 2, 2023 · 1 comment
Open

add location to DynamicProxy #6346

sanderr opened this issue Aug 2, 2023 · 1 comment
Labels
compiler enhancement New feature or request suggestion Suggest a change, open to discussion usability This ticket is related to improved usability

Comments

@sanderr
Copy link
Contributor

sanderr commented Aug 2, 2023 •
edited
Loading

Adding a location property/method to DynamicProxy would allow plugins to raise more useful error messages when a specific model instance is involved. Alternatively/additionally we could go for an identity method that returns more or less what repr returns for the AST node, e.g. 'lsm::ServiceEntityBindingV2 (instantiated at /tmp/tmppvbkd07y/main.cf:5)'. This contains both the location and the value/type of the object.

Inspired by inmanta/lsm#627
@sanderr sanderr added enhancement New feature or request suggestion Suggest a change, open to discussion compiler usability This ticket is related to improved usability labels Aug 2, 2023
@sanderr
Copy link
Contributor Author

sanderr commented Aug 2, 2023

Related to this Slack discussion, especially this comment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
compiler enhancement New feature or request suggestion Suggest a change, open to discussion usability This ticket is related to improved usability
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@sanderr