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

AbapLint Backend fails his own checks #48

Closed
ed-holland opened this issue Jun 29, 2020 · 6 comments
Closed

AbapLint Backend fails his own checks #48

ed-holland opened this issue Jun 29, 2020 · 6 comments

Comments

@ed-holland
Copy link
Collaborator

image

There are more than 16 pages of errors.

@larshp
Copy link
Member

larshp commented Jun 30, 2020

in #49 (comment) it reports unused_variables but not on command line, this is within the same class

unknowns: what is the max dependency depth configured in your system?

@ed-holland
Copy link
Collaborator Author

It is not defined. But also with level 2 (which takes ages) there still are problems.
Are you not having this issue?

@ed-holland
Copy link
Collaborator Author

I've reran SCI on the package as there where many changes made to Linter/Dependencies (Thanks @larshp).
SCI lint configuration was updated from AG and saved.
image

When running AbapLint on the repository, no issues are found.

abaplint-backend from 2020-07-19 13:25:38
abaplint version: 2.38.1

@ed-holland
Copy link
Collaborator Author

I've rechecked with the latest Lint version 2.39.21 and abaplint.json. The list has changed but there are still differences.

image

Running Linter externally, has no issues.
image

@ed-holland
Copy link
Collaborator Author

With the latest AbapLint 2.48.0, the differences between SCI and abaplint are reduced.
image

@ed-holland
Copy link
Collaborator Author

Closing this one. Not so important.

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