codefori / vscode-rpgle

RPGLE language tools for VS Code
MIT License
39 stars 20 forks source link

Feature/rename_provider #266

Closed worksofliam closed 11 months ago

worksofliam commented 1 year ago

Changes

Initial work to start refactoring for RPG (**free only)

Todo

Checklist

worksofliam commented 1 year ago

Depends on #265 for cache.ts rewrite (js -> ts).

worksofliam commented 11 months ago

@chrjorgensen You are amazing for your review. I missed a whole bunch of logic for definition subfield references and now it is solved (with test cases!). Thanks for your example, which I went ahead and borrowed as you will see in the commit.

Please try again and let me know what you think.

chrjorgensen commented 11 months ago

@worksofliam I just did some new tests:

And the spelling mistake refereneceByOffset has not yet been fixed...

worksofliam commented 11 months ago

@chrjorgensen Errors solved and spelling fixed!

chrjorgensen commented 11 months ago

@worksofliam

@chrjorgensen Errors solved and spelling fixed!

Did you miss something? I get the following errors now when typing F2:

image

image

worksofliam commented 11 months ago

@chrjorgensen I am sorry. I didn't do a push 🙄 Try again!

chrjorgensen commented 11 months ago

@worksofliam Approved - merge whenever you feel for it! 😃