Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix undefined method error in facts #555

Merged

Conversation

wiebe
Copy link
Contributor

@wiebe wiebe commented Jul 21, 2020

Pull Request (PR) description

Prevents fact errors when pip/virtualenv installation is broken and adds version fact for pip2 and pip3.

This Pull Request (PR) fixes the following issues

Fixes #527

Partly fixes voxpupuli#527.

Signed-off-by: Wiebe Verweij <wiebe.verweij@moxio.com>
Partially fixes voxpupuli#527

Signed-off-by: Wiebe Verweij <wiebe.verweij@moxio.com>
@wiebe wiebe force-pushed the fix_undefined_method_error_in_facts branch from 8ad245a to 99ab8cc Compare July 21, 2020 11:31
@bastelfreak
Copy link
Member

thanks for the PR!

@bastelfreak bastelfreak added enhancement New feature or request bug Something isn't working and removed enhancement New feature or request labels Sep 12, 2020
@bastelfreak bastelfreak merged commit afc4022 into voxpupuli:master Sep 12, 2020
bastelfreak added a commit to bastelfreak/puppet-python that referenced this pull request Sep 13, 2020
voxpupuli#555 introduced new
tests. In the same time we adjusted our test setup on the master branch
and voxpupuli#555 wasn't rebased before merging. This PR fixes the tests.
@bastelfreak bastelfreak mentioned this pull request Sep 13, 2020
russellshackleford pushed a commit to russellshackleford/puppet-python that referenced this pull request Sep 28, 2020
voxpupuli#555 introduced new
tests. In the same time we adjusted our test setup on the master branch
and voxpupuli#555 wasn't rebased before merging. This PR fixes the tests.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"undefined method []" in pip_version and virtualenv_version facts
2 participants