-
Notifications
You must be signed in to change notification settings - Fork 21
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
Feature/rpgle_2023_oct #265
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@worksofliam Looking good, the ENUMs and constants are recognized and shown in Outline and Content Assist. 👍
I have only a few issues:
- How do I test the WHEN tokens being recognized?
- I would expect the subitems in a qualified ENUM to be shown in Content Assist, when I type the name of the ENUM and a dot and Ctrl-Space - but they are not in the list? This is also a problem for subfields in a data structure, so maybe it is an old problem? If I don't qualify the ENUM or data structure, the subitems appear in the Content Assist list...
@chrjorgensen I added two tests cases for the new |
@worksofliam I did a fresh install of VS Code, Code for IBM i and RPGLE extension in a new Windows profile. I don't know what going on here... not like your experience at all...??? |
@chrjorgensen okay I see what's going on. Control+space will only show in scope references. You will only see subfields when you press dot after the structure. |
@worksofliam Finally found the setting, that prohibited the pop-up when entering the dot after the structure:
After removal of this setting, the subfields are shown after entering the dot character. Will review your code tomorrow. |
Totally cool! Weird that it was off though. |
Probably me that have turned if off at some point - annoyed by the pop-ups... 😆 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@worksofliam Looking good - only a few issues...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@worksofliam Some final issues - when fixed we can approve and merge! 🚀
Co-authored-by: Christian Jorgensen <[email protected]>
@chrjorgensen Issues resolved. Over to you for approval, and if you want to.. merge 😄 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@worksofliam Great - let's approve and merge! 🚀
@worksofliam I don't have write access and can't merge... 😭 |
Changes
WHEN-IS
andWHEN-IN
Checklist
console.log
s I added