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.

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

Support for Corinna keywords #299

Open
davorg opened this issue Jul 18, 2024 · 4 comments
Open

Support for Corinna keywords #299

davorg opened this issue Jul 18, 2024 · 4 comments

Comments

@davorg
Copy link

davorg commented Jul 18, 2024

I raised an issue on Perl::Metrics::Simple because it doesn't recognise that method defines a subroutine. And it turns out that the problem is PPI not supporting method (and, also, class and field).

How hard would it be to support these new keywords? I know nothing about the internals of PPI, but I'd be willing to learn if it would help get this added.

@oalders
Copy link
Collaborator

oalders commented Jul 18, 2024

@davorg please see #273 to get an overview of the issue.

@davorg
Copy link
Author

davorg commented Jul 18, 2024

So what I'm seeing is that the PPI team was thinking about this issue even before perlclass was a thing. Which is great :-)

Feel free to ping me if you think there's anything useful I can add.

@wchristian
Copy link
Member

I've been wanting to put more time into this recently, so this is a good push for me. :)

The one thing that could help would be reading over this issue to see how you feel, if any of it makes sense. Otherwise i'll end up pushing forward along that line soon. :) #280

@davorg
Copy link
Author

davorg commented Jul 18, 2024

Will do. Thanks.

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

No branches or pull requests

3 participants