Skip to content

feat: add ABAP Doc comments to all public methods and interfaces#45

Merged
MagPasulke merged 5 commits into
mainfrom
feat/add-abapdoc
May 24, 2026
Merged

feat: add ABAP Doc comments to all public methods and interfaces#45
MagPasulke merged 5 commits into
mainfrom
feat/add-abapdoc

Conversation

@MagPasulke
Copy link
Copy Markdown
Owner

Adds abapdoc ("!) comments to all public methods and interface methods that triggered the new abapdoc abaplint rule.

⚠️ Please sync to SAP system via abapGit and run ABAP Unit tests before merging.

Fixes 18 abapdoc abaplint violations across 13 files:
- auth: zasis_cx_no_auth constructor, zasis_if_auth_checker (4 methods)
- bo: zasis_cl_interpreter constructor, zasis_cl_ruleset constructor,
  zasis_cl_ruleset_factory (3 methods), zasis_cx_ruleset_ui constructor,
  zasis_if_customlogic execute, zasis_if_customlogic_resolver resolve,
  zasis_if_ev_producer_resolver resolve, zasis_if_event_producer on_item_interpreted,
  zasis_if_interpreter execute
- utils: zasis_cl_class_validator check_implements, zasis_cx_exc constructor
@MagPasulke MagPasulke marked this pull request as ready for review May 24, 2026 21:20
Replace minimal shortText-only comments with full ABAP Doc blocks
following the official @parameter | desc and @Raising | desc syntax.
All 13 affected files now document every parameter and exception.
@MagPasulke MagPasulke merged commit 71ed870 into main May 24, 2026
1 check passed
@MagPasulke MagPasulke deleted the feat/add-abapdoc branch May 24, 2026 21:27
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