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

Supports Calling Procs w/ Secondary Entry Point #50

Open
wants to merge 2 commits into
base: development
Choose a base branch
from

Conversation

montymxb
Copy link
Contributor

Resolves #33 by expanding the grammar to accept NamedElements when considering callable things. The checking for declarations is setup in a new validation rule + associated validation tests.

Also took a moment to refactor a bunch of incorrectly generated compiler message fields in the pli-codes file. Many of these were supposed to be parametric, but the extraction & conversion scripts I wrote up didn't take into account some funky spacing. Those cases have been corrected to accept args accordingly for better message output.

Copy link

github-actions bot commented Feb 20, 2025

PR Preview Action v1.6.0

🚀 View preview at
https://zowe.github.io/zowe-pli-language-support/pr/pr-50/

Built to branch previews at 2025-02-25 08:58 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

Signed-off-by: Benjamin Friedman Wilson <[email protected]>
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

Successfully merging this pull request may close these issues.

1 participant