Skip to content

Adding O_spec feature#487

Closed
Monikacopilot2 wants to merge 3 commits into
codefori:mainfrom
Monikacopilot2:O-spec_feature
Closed

Adding O_spec feature#487
Monikacopilot2 wants to merge 3 commits into
codefori:mainfrom
Monikacopilot2:O-spec_feature

Conversation

@Monikacopilot2

Copy link
Copy Markdown

Added comprehensive O-spec (Output Specification) parsing support to the fixed-form RPG parser in language/models/fixed.js.
Changes Made
New Function: parseOLine()
Implemented a complete parser for fixed-form RPG Output specifications that extracts all standard O-spec fields according to IBM i RPG column positions

@mounigopisetti1162

Copy link
Copy Markdown

I tested the code, and it’s working as expected. I’m able to see the O-specs.

@bobcozzi

Copy link
Copy Markdown
Collaborator

I've added basic O and I spec "Ruler" support to the RPGLE extension. To support O and I better.

@worksofliam

Copy link
Copy Markdown
Member
  1. There are not enough test cases for O-spec support. Need a lot more, since there are lots of edge cases.
  2. The bigger discussion that is needed is I and O spec support in general. I did start on I support and made good progress, but I never finished it. (Branch feature/i_spec)

@worksofliam worksofliam left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

see comment above

@Monikaibmi15

Copy link
Copy Markdown

Added testcase for O-spec for support.

@worksofliam

Copy link
Copy Markdown
Member

@Monikaibmi15 If you could fix the conflicts, that would be great

@worksofliam worksofliam left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

fix the conflicts and we're good

@supravi96

Copy link
Copy Markdown

@worksofliam This PR had a significant number of merge conflicts, so we’ve created a new PR with all conflicts resolved and the latest updates incorporated to make the review process smoother.

Please refer to the new PR for review #507

@worksofliam worksofliam closed this May 6, 2026
@venky225

Copy link
Copy Markdown
Contributor

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.

7 participants