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

Update abaplint.json, set folder #92

Merged
merged 2 commits into from
Aug 8, 2024
Merged

Update abaplint.json, set folder #92

merged 2 commits into from
Aug 8, 2024

Conversation

larshp
Copy link
Contributor

@larshp larshp commented Aug 8, 2024

I've set the configuration to https://github.com/abaplint/abaplint/blob/main/docs/getting_started.md

when its green, then additional rules can be enabled

@fidley fidley merged commit bc7b4a7 into fidley:master Aug 8, 2024
2 of 3 checks passed
@fidley
Copy link
Owner

fidley commented Aug 8, 2024

Thanks, I've haven't finished the customization of it, in fact I was wondering if with this very bad coding I have inside, is it possible still. As I don't want to make non-backward compatible changes in the names of the parameters and methods... but let me spend some time on this.

@larshp
Copy link
Contributor Author

larshp commented Aug 8, 2024

its very strict in the default setup, with everything enabled, even some stuff that I'd typically not use

it will get there, just need a few things fixed, like abaplint/abaplint#3388

https://abaplint.app/stats/fidley/falv/disabled_rules will update tomorrow, so you can take a look at which rules you'd like enabled

@larshp larshp deleted the patch-1 branch August 8, 2024 08:01
@fidley
Copy link
Owner

fidley commented Aug 8, 2024

Thanks, I'll take a look tomorrow then

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