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

(CONT-173) - Updating deprecated facter instances #531

Merged
merged 1 commit into from
Oct 20, 2022

Conversation

jordanbreen28
Copy link
Contributor

Prior to this PR, this module contained instances of Facter::Util::Resolution.exec and Facter::Util::Resolution.which, which are deprecated. This PR aims to replace these exec helpers with their supported Facter::Core::Execution counterparts.

This PR:

  • Replaces all Facter::Util::Resolution instances with corresponding Facter::Core::Execution exec helpers

@jordanbreen28 jordanbreen28 requested a review from a team as a code owner October 13, 2022 15:35
@jordanbreen28 jordanbreen28 force-pushed the cont-173/main/update_deprecated_facter branch from df10faa to 9bec045 Compare October 17, 2022 13:13
@jordanbreen28 jordanbreen28 force-pushed the cont-173/main/update_deprecated_facter branch 3 times, most recently from 2b72c4a to be33af6 Compare October 20, 2022 13:00
Prior to this PR, this module contained instances of Facter::Util::Resolution.exec and Facter::Util::Resolution.which, which are deprecated.
This PR aims to replace these exec helpers with their supported Facter::Core::Execution counterparts.

This PR:

- Replaces all Facter::Util::Resolution instances with corresponding Facter::Core::Execution exec helpers

Fixed Spec Test Failures
@jordanbreen28 jordanbreen28 force-pushed the cont-173/main/update_deprecated_facter branch from be33af6 to 19afda2 Compare October 20, 2022 13:02
Copy link
Contributor

@chelnak chelnak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look good and it makes sense to refactor the fact definition.

We should keep an eye out for issues to just to be safe.

@chelnak chelnak merged commit 7c88fb0 into main Oct 20, 2022
@chelnak chelnak deleted the cont-173/main/update_deprecated_facter branch October 20, 2022 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants