larshp / abapOpenChecks

Open source checks for SAP Code Inspector / ABAP Test Cockpit
https://docs.abapopenchecks.org
MIT License
270 stars 79 forks source link

Some checksums missing for Baseline #1079

Closed se38 closed 2 years ago

se38 commented 2 years ago

I've found some more checks, where the checksum is missing for baseline (like in issue #647 and PR #677 )

IF in IF, can easily be reduced  001 - IF in IF 
Commented code                   014 - Commented code  
EXPORTING can be omitted         030 - EXPORTING can be omitted  
Use condense( )                  045 - Use expressions  
Found a suspicious comment: ...  070 - Find comment markers  

Will try to correct them this week.

se38 commented 2 years ago

If someone know how to add checksums in checks 14 and 70 please let me know. I think this isnt't possible because comments are no statements and therfore there is no statement "position".

For the other three checks I've created a pull request.

se38 commented 2 years ago

Closed with PR #1080