Project

General

Profile

Actions

action #97097

open

coordination #96596: [qe-core][CI] CI/CD and Coding style improvements

[qe-core][CI] Add syntax checkers to the CI to verify the parameter parsing (move away from my $self = shift)

Added by dvenkatachala over 2 years ago. Updated 8 months ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
Enhancement to existing tests
Target version:
-
Start date:
2021-08-18
Due date:
% Done:

0%

Estimated time:
Difficulty:

Description

  • Syntax checkers for coding style

    In an existing Perl module or a new file is created, avoid the mix between my $self = shift and my ($self) = @_ for parameter parsing and the suggestion is to stick to my ($self) = @.
    Implement CI check to verify the parameter parsing in the files.

AC's for each item

  • AC1: Each subtask is proposed as an RFC in the os-autoinst-distri-opensuse repo.
  • AC2: There's a corresponding entry in the contributing document.
  • AC3: A CI check to verify parameter parsing in the newly created/edited files similar to [PR]
Actions

Also available in: Atom PDF