Feature #10966

Add bios information to facter

Added by Tomas Edwardsson 6 months ago. Updated 2 months ago.

Status:Tests Insufficient Start date:11/19/2011
Priority:Normal Due date:
Assignee:- % Done:

0%
Category:library
Target version:-
Keywords: Affected Facter version:
Branch:
Votes: 0

Description

Patch and pull request at:

https://github.com/puppetlabs/facter/pull/81

History

#1
Updated by Ken Barber 6 months ago

  • Category set to library
  • Status changed from Unreviewed to Accepted
  • Target version set to 186

Hi Tomas,

Thanks for this …

Can I suggest we use ‘biosversion’ without an underscore? It fits the current convention.

Also – the pull request is no longer merge-able. Can you please rebase this?

#2
Updated by Daniel Pittman 4 months ago

  • Status changed from Accepted to Code Insufficient

The pull request has merge conflicts and unrelated commits in it at the moment; we have asked the author to update that.

#3
Updated by Ken Barber 4 months ago

  • Status changed from Code Insufficient to Tests Insufficient

Pull request is here:

https://github.com/puppetlabs/facter/pull/144

We’ll need tests.

#4
Updated by Daniel Pittman 2 months ago

  • Target version deleted (186)

Also available in: Atom PDF