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

Additional Linter Options #257

Open
js-mason opened this issue Sep 29, 2023 · 1 comment
Open

Additional Linter Options #257

js-mason opened this issue Sep 29, 2023 · 1 comment
Labels

Comments

@js-mason
Copy link

I have one linter config setup for our shop and share it between our developers in order to keep our code consistent.
This works, but I see an opportunity for additional code formatting options in the rpgle linter. Here are several suggestions...

- Include space after "(" and before ")" ; 
         example:   Int(10)    would be converted to   Int( 10 )

-  Include space [ before | after | nospace ] with the parameter separator ":" character
         example:    nospace     Procedure( Parm1:Parm2:Parm3 )
                            before         Procedure( Parm1 :Parm2 :Parm3 )
                            after            Procedure( Parm1: Parm2: Parm3 )

-  Require or remove Const keyword on Dcl-C statements
          example:     require      Dcl-C  SOMECONSTANT  Const( 'XYZ' );
                              remove      Dcl-C  SOMECONSTANT   'XYZ';
@js-mason js-mason added the idea label Sep 29, 2023
@worksofliam worksofliam transferred this issue from codefori/vscode-ibmi Sep 29, 2023
@chrjorgensen
Copy link
Contributor

I suggest adding an additional option surround for the parameter separator:

-  Include space [ before | after | surround | nospace ] with the parameter separator ":" character
         example:    nospace     Procedure( Parm1:Parm2:Parm3 )
                     before      Procedure( Parm1 :Parm2 :Parm3 )
                     after       Procedure( Parm1: Parm2: Parm3 )
                     surround    Procedure( Parm1 : Parm2 : Parm3 )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants