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)
0%
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 about 3 years ago
- Parent task changed from #95161 to #96596
Updated by dvenkatachala about 3 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 about 3 years ago
- Tracker changed from coordination to action
Updated by dvenkatachala about 3 years ago
- Project changed from 175 to openQA Tests
Updated by okurz about 3 years ago
- Category set to Enhancement to existing tests
Updated by punkioudi about 3 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 about 3 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 about 3 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 about 2 years ago
- Status changed from In Progress to New
- Assignee deleted (
dvenkatachala)
I guess this is not in progress anymore
Updated by szarate about 1 year ago
Updated by slo-gin 3 months ago
This ticket was set to Normal priority but was not updated within the SLO period. Please consider picking up this ticket or just set the ticket to the next lower priority.