Skip to content

Conversation

apozharski
Copy link
Contributor

This extends the work in #88 to also improve parsing of property, events, and methods blocks.

WIP until tests are updated.

@dklilley
Copy link
Member

Hi @apozharski - I wanted to check back to see if there are any plans for this PR? If not, I will close this PR.
Thanks!

@apozharski
Copy link
Contributor Author

Ah, my apologies, I got very busy and this fell off the back of the TODO list. Thanks for the reminder. Let me take a look this week at how close this was to something that could be merged, which I presume at least will involve getting tests in working order.

@dklilley
Copy link
Member

No worries! Thank you!

@apozharski apozharski force-pushed the fixing-attribute-parsing-everywhere branch from 203681d to bcaa109 Compare November 28, 2024 12:45
@apozharski apozharski force-pushed the fixing-attribute-parsing-everywhere branch from bcaa109 to 9756824 Compare November 28, 2024 12:45
@apozharski
Copy link
Contributor Author

I added some tests and fixed existing ones. I believe this now actually addresses #95 and #92.

@apozharski apozharski changed the title [WIP] fixed attribute parsing in properties methods and events fixed attribute parsing in properties methods and events Nov 29, 2024
@dklilley dklilley self-assigned this Dec 4, 2024
@dklilley dklilley added the CLA signed Indicates that the requester has signed the Contributor License Agreement label Dec 4, 2024
@dklilley
Copy link
Member

dklilley commented Dec 4, 2024

These changes look good to me! I copied this over into the MATLAB extension for VS Code and quickly verified that the specific cases (#92 and #95) look good!

Thanks for working on this!

@dklilley dklilley merged commit d7fa696 into mathworks:master Dec 4, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA signed Indicates that the requester has signed the Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants