Actions
action #97097
opencoordination #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)
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]
Updated by dvenkatachala over 2 years ago
- Parent task changed from #95161 to #96596
Updated by dvenkatachala over 2 years ago
- Subject changed from Add syntax checkers to the CI to propose changes over old coding style to [qe-core][CI] Add syntax checkers to the CI to propose changes over old coding style
Updated by dvenkatachala over 2 years ago
- Tracker changed from coordination to action
Updated by dvenkatachala over 2 years ago
- Project changed from 175 to openQA Tests
Updated by punkioudi over 2 years ago
Hi Dee, I am not sure if this PR : https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/13323 serves the purpose of the CI story. Plus, this ticket doesn't have a description of what you are trying to achieve within this ticket :)
Updated by dvenkatachala over 2 years ago
- Subject changed from [qe-core][CI] Add syntax checkers to the CI to propose changes over old coding style to [qe-core][CI] Add syntax checkers to the CI to verify the parameter parsing
- Description updated (diff)
Updated by szarate over 2 years ago
- Subject changed from [qe-core][CI] Add syntax checkers to the CI to verify the parameter parsing to [qe-core][CI] Add syntax checkers to the CI to verify the parameter parsing (move away from my $self = shift)
Updated by szarate over 1 year ago
- Status changed from In Progress to New
- Assignee deleted (
dvenkatachala)
I guess this is not in progress anymore
Actions