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

dynpro checks: CHAIN indentation false positive #3511

Closed
mbtools opened this issue Dec 19, 2024 · 8 comments
Closed

dynpro checks: CHAIN indentation false positive #3511

mbtools opened this issue Dec 19, 2024 · 8 comments

Comments

@mbtools
Copy link
Collaborator

mbtools commented Dec 19, 2024

CHAIN. block should be indented

CHAIN. 
  FIELD input3. 
  FIELD input4. 
  FIELD input5. 
  MODULE check_chain ON CHAIN-REQUEST. 
ENDCHAIN. 

https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/index.htm?file=dynpchain.htm

@larshp
Copy link
Member

larshp commented Dec 20, 2024

unable to reproduce, what is the full example?

@larshp
Copy link
Member

larshp commented Dec 20, 2024

screen example added in playground

@mbtools
Copy link
Collaborator Author

mbtools commented Dec 20, 2024

PROCESS BEFORE OUTPUT.
  MODULE pbo_1001.

PROCESS AFTER INPUT.
  CHAIN.
    FIELD test.
  ENDCHAIN.
/zfoo.prog.screen_0100.abap[6, 5]: Indentation problem, expected 2 spaces(indentation)

@larshp
Copy link
Member

larshp commented Dec 23, 2024

@mbtools try again, 2.113.85 published

@mbtools
Copy link
Collaborator Author

mbtools commented Dec 29, 2024

Thank you, it works... in other repos.

image

On https://github.com/Marc-Bernard-Tools/MBT-System-Monitor, I tried new PRs and commits but it is stuck on 2.113.78. 🤷 How do I get it to 2.113.85?

@larshp
Copy link
Member

larshp commented Dec 30, 2024

@mbtools try again

@mbtools
Copy link
Collaborator Author

mbtools commented Dec 30, 2024

almost. it now runs 2.113.85 but getting "Main file must have specific contents: Report must begin with REPORT or PROGRAM"

the error is also there if the main program is included in the commit.

image

@larshp
Copy link
Member

larshp commented Jan 3, 2025

@mbtools try again

@mbtools mbtools closed this as completed Jan 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants