Skip to content

Disable unused methods and variables checks#123

Closed
larshp wants to merge 1 commit into
MagPasulke:mainfrom
larshp:patch-2
Closed

Disable unused methods and variables checks#123
larshp wants to merge 1 commit into
MagPasulke:mainfrom
larshp:patch-2

Conversation

@larshp

@larshp larshp commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

disable these for now, to get it green

see abaplint/abaplint#4008 (comment)

disable these for now, to get it green
@MagPasulke

Copy link
Copy Markdown
Owner

disable these for now, to get it green

see abaplint/abaplint#4008 (comment)

@larshp
hmm why would I?
I marked them as called ##CALLED, since they are in fact called by RAP framework, see:
https://github.com/MagPasulke/abap-string-interpreter/blob/main/src/bo/zbp_asis_i_ruleset.clas.locals_imp.abap#L5

static code analysis passes and I still get the feedback everywhere else.

@larshp larshp closed this Jun 18, 2026
@larshp larshp deleted the patch-2 branch June 18, 2026 12:04
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