Project

General

Profile

Actions
Copy link

action #9494

closed

Comment (and refactor if possible) main.pm

Added by RBrownSUSE over 8 years ago. Updated over 7 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
okurz
Category:
Enhancement to existing tests
Target version:
-
Start date:
2015-11-11
Due date:
% Done:

0%

Estimated time:
Difficulty:

Description

To easy contribution of new tests, a more understandable main.pm is needed so people not familiar with openqa.(suse.de/opensuse.org) can introduce new tests in the right place.

https://github.com/os-autoinst/os-autoinst-distri-opensuse/commit/7c9885ef167a5e853a36bb0100490704258a0dee is already a step in the right direction.

Hide closed

Checklist

  • SLE
  • Leap
  • TW
Actions
Copy link
 #1

Updated by RBrownSUSE over 8 years ago

  • Checklist item changed from to [ ] SLE, [ ] Leap, [ ] TW
  • Target version deleted (154)
Actions
Copy link
 #2

Updated by okurz over 7 years ago

  • Status changed from New to Resolved
  • Assignee set to okurz

A big cleanup has been done with

commit cbb3575
Author: Oliver Kurz okurz@suse.de
Date: Fri Jan 8 22:57:35 2016 +0100

Separate opensuse and sle12 in directories rather than branches

and also

commit 39ba846
Author: Oliver Kurz okurz@suse.de
Date: Thu Jun 23 15:27:02 2016 +0200

Extract all common code from both main.pm into a common file

I guess from here there are no more low hanging fruits unless we provide something which is way more elaborate and not a beginner task. Let's close this issue.

Actions
Copy link

Also available in: Atom PDF