Skip to content

updated actions for rpg and rpgle member file extensions#3113

Open
supravi96 wants to merge 4 commits intocodefori:masterfrom
supravi96:update-rpgle/rpg-actions
Open

updated actions for rpg and rpgle member file extensions#3113
supravi96 wants to merge 4 commits intocodefori:masterfrom
supravi96:update-rpgle/rpg-actions

Conversation

@supravi96
Copy link
Copy Markdown

@supravi96 supravi96 commented Mar 15, 2026

Changes

As of now code for ibm i returns rpgle actions for when user invokes get actions tool to fetch available actions for rpg member file. These 4 commands are only valid for .rpgle. This pr will update the code for ibmi to return 4 valid rpgle commands and 1 valid .rpg command . test cases are added for .rpg command as well.

How to test this PR

Checklist

  • have tested my change
  • have created one or more test cases
  • updated relevant documentation
  • Remove any/all console.logs I added
  • have added myself to the contributors' list in CONTRIBUTING.md

Copy link
Copy Markdown
Member

@sebjulliand sebjulliand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, this won't do.

  • CRTPGM and CRTSRVPGM are meant to be GLOBAL - these are not commands related to RPGLE
  • Why add support out-of-the-box for RPG36 and RPG38?

The changes on CRTBNDRPG and CRTRPGMOD are fine.

@supravi96 supravi96 marked this pull request as draft March 18, 2026 05:27
@supravi96 supravi96 requested a review from sebjulliand March 18, 2026 11:03
@supravi96 supravi96 marked this pull request as ready for review March 18, 2026 11:04
@supravi96
Copy link
Copy Markdown
Author

@sebjulliand package.json is now updated to return actions only for .rpg. and the global actions that were changed to RPGLE are now updated back to GLOBAL. Also test cases are added for .rpg action.

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.

2 participants