QA (public) » openQA Project (public)

action #179975

Updated by tinita 23 days ago

## Motivation 

 We use a common code style for all our perl code. 

 [TooMuchCode::ProhibitUnusedImport](https://metacpan.org/pod/Perl::Critic::Policy::TooMuchCode::ProhibitUnusedImport) [TooMuchCode](https://metacpan.org/pod/Perl::Critic::TooMuchCode) can check for unused module imports. imports among other policies. 
 It also has [ProhibitExtraStricture](https://metacpan.org/pod/Perl::Critic::Policy::TooMuchCode::ProhibitExtraStricture) which seems better than our own implementation. 

 ## Acceptance criteria 
 * **AC1:** openQA perlcritic checks cover all directories with perl code 
 * **AC2:** We use the policies from `Perl::Critic::TooMuchCode` 
 * **AC3:** Superfluous implementations in os-autoinst-common are removed 

Back