Project

General

Profile

Actions

action #117118

closed

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

[qe-core] CI/CD and Coding style improvements - Forbid usage of fgrep and egrep

Added by szarate over 1 year ago. Updated over 1 year ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Refactor/Code Improvements
Target version:
Start date:
2022-09-23
Due date:
% Done:

0%

Estimated time:
Difficulty:
Sprint:
QE-Core: October Sprint (Sep 28 - Oct 26)

Description

As a follow up to https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/15547

Since we're having multiple styles of coding and tools, often we end up with a mix, and in the case of grep, it seems that the time for unification has arrived:

fgrep and egrep are now deprecated, so to avoid warnings, we should switch to using extended mode,

Acceptance criteria

  1. Style checks t/01_style.t also verify that neither egrep nor fgrep are being used.
  2. RFC to CONTRIBUTING.md with the above rule in has been created (Or merged in case os-autoinst-distri-opensuse#15547 has been already merged)

Suggestions

Evaluate how #96675 was done

Actions #1

Updated by szarate over 1 year ago

  • Tracker changed from coordination to action
Actions #2

Updated by szarate over 1 year ago

  • Description updated (diff)
Actions #3

Updated by szarate over 1 year ago

  • Sprint set to QE-Core: October Sprint (Sep 28 - Oct 26)
  • Tags set to qe-core-october-sprint
  • Description updated (diff)
  • Category changed from New test to Refactor/Code Improvements
  • Status changed from New to Workable
  • Target version set to QE-Core: Ready
Actions #4

Updated by VANASTASIADIS over 1 year ago

  • Assignee set to VANASTASIADIS
Actions #5

Updated by VANASTASIADIS over 1 year ago

  • Status changed from Workable to Blocked

pr ready: https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/15704

waiting for the related pr to be merged first, as the opposite would lead to the ci failing.

Actions #6

Updated by VANASTASIADIS over 1 year ago

  • Status changed from Blocked to Resolved
Actions

Also available in: Atom PDF