Add nginx_version fact

[jyaworski]

No description provided.

[3flex]

Please add a test for this. Can use puppetlabs/puppetlabs-mysql@3ce8ad3 as a template.

[jyaworski]

@3flex done. Thanks.

[jyaworski]

Hmm. I'm not familiar with testing this particular fact type. Let me look into it; the test fails, but the fact works.

[jyaworski]

OK, I got it. I forgot to stub :which.

[3flex]

Looks good. Hopefully obvious, but did you confirm it works on a real machine?

[jyaworski]

Using it in production now, actually.

On Jan 29, 2016, at 4:44 PM, Matthew Haughton notifications@github.com wrote:

Looks good. Hopefully obvious, but did you confirm it works on a real machine?

—
Reply to this email directly or view it on GitHub.

[3flex]

@jyaworski I'm proposing this be reverted (see #760) because it's not supported by earlier versions of Facter which are still supported by this module.

Are you able to make a version that is compatible back to Facter 1.7? If so, please submit a new PR. Thanks!