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

(SUP-2848) st0361a_uploading_facts convert to ruby #127

Merged
merged 1 commit into from
Dec 20, 2021

Conversation

MartyEwings
Copy link
Collaborator

Removes bash and power shell versions
adds spec tests

@MartyEwings
Copy link
Collaborator Author

@m0dular are you able to approve? ideally we wouldn't "Shell out" but given this is run infrequently, and the general improvements we have since moving from BASH and powershell this is a positive step

@MartyEwings MartyEwings force-pushed the SUP-2848 branch 2 times, most recently from d46fece to 6889ae8 Compare December 20, 2021 11:54
Copy link
Contributor

@jarretlavallee jarretlavallee left a comment

Choose a reason for hiding this comment

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

👍

@MartyEwings MartyEwings merged commit 6b459e3 into puppetlabs:main Dec 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants