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

code-pal, "a set of rules to guarantee the Clean ABAP adherence" #541

Closed
larshp opened this issue Dec 16, 2021 · 4 comments
Closed

code-pal, "a set of rules to guarantee the Clean ABAP adherence" #541

larshp opened this issue Dec 16, 2021 · 4 comments
Labels
documentation Check documentation (markdown)

Comments

@larshp
Copy link
Contributor

larshp commented Dec 16, 2021

ref 0f34fa8

"this tool contains a set of rules to guarantee the Clean ABAP adherence"

so reading this, it means that if the rules of code-pal is green, my code follows https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md ?

same with "based on Clean ABAP", this means all rules are based on Clean ABAP?

@larshp larshp added the bug Something isn't working correctly label Dec 16, 2021
@lucasborin
Copy link
Member

so reading this, it means that if the rules of code-pal is green, my code follows https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md ?

Not at all, there are some rules we cannot guarantee via static checks.

same with "based on Clean ABAP", this means all rules are based on Clean ABAP?

IMO, yes, they are. Code Pal follows what is defined in the Clean ABAP style guide. Please, report if you find something going in the wrong direction. But yeah, as you mentioned, Code Pal does not guarantee that everything is covered. How about changing the word guarantee to support?

Based on the Clean ABAP, this tool contains a set of rules to support Clean ABAP adherence.

@lucasborin lucasborin removed the bug Something isn't working correctly label Dec 20, 2021
@larshp
Copy link
Contributor Author

larshp commented Dec 20, 2021

"support" is good, its not misleading...

@larshp
Copy link
Contributor Author

larshp commented Dec 20, 2021

lucasborin added a commit that referenced this issue Dec 20, 2021
@bjoern-jueliger-sap bjoern-jueliger-sap added the documentation Check documentation (markdown) label Mar 16, 2022
@larshp
Copy link
Contributor Author

larshp commented May 5, 2022

so, the same misleading information now made it into https://blogs.sap.com/2022/05/05/how-to-enable-clean-code-checks-for-abap/

bjoern-jueliger-sap added a commit to bjoern-jueliger-sap/code-pal-for-abap that referenced this issue Aug 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Check documentation (markdown)
Projects
None yet
Development

No branches or pull requests

3 participants