Skip to content
This repository has been archived by the owner on Apr 7, 2022. It is now read-only.

Filedump artifactor plugin revamp #2554

Merged
merged 2 commits into from
Dec 18, 2015
Merged

Filedump artifactor plugin revamp #2554

merged 2 commits into from
Dec 18, 2015

Conversation

mfalesni
Copy link
Contributor

Make filedump more flexible.

{{pytest: cfme/tests/configure/test_docs.py -v }}

@dajoRH dajoRH added the lint-ok label Dec 17, 2015
Milan Falešník and others added 2 commits December 18, 2015 16:23
* Support putting an unlimited number of files in the report
* Make all file-outputting things use filedump
* Remove the standalone softassert artifactor plugin since it is replaced by the multiple file functionality of the new filedump
* Files can be grouped using group_id, because one line per file is too wasting but having all files without line breaks would looks awful
* riggerlib plugin now holds safe nested supprt for fire_hook and so
  importing the art_client is no longer needed
@dajoRH
Copy link
Contributor

dajoRH commented Dec 18, 2015

Lint report for commit 3ace41f:
:godmode: All seems good! 🍰 👊 🍪
CFME QE Bot

@psav psav added the reviewed label Dec 18, 2015
psav added a commit that referenced this pull request Dec 18, 2015
Filedump artifactor plugin revamp
@psav psav merged commit 29e44f0 into ManageIQ:master Dec 18, 2015
@mfalesni mfalesni deleted the fieldump-revamp branch December 18, 2015 17:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants