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

(maint) add a little bit of verbosity to deploy_signed_repos_to_s3 #1273

Merged
merged 1 commit into from
Oct 28, 2024

Conversation

e-gris
Copy link
Collaborator

@e-gris e-gris commented Oct 23, 2024

  • Make deploy_signed_repos_to_s3 print a message that it is doing the s3sync
  • Refactor messy code around it
  • Make rspec work

Please add all notable changes to the "Unreleased" section of the CHANGELOG.

@e-gris e-gris requested a review from a team as a code owner October 23, 2024 23:38
@e-gris e-gris force-pushed the deploy_to_s3_feedback branch 2 times, most recently from 396115e to 444239e Compare October 23, 2024 23:42
@e-gris e-gris force-pushed the deploy_to_s3_feedback branch 2 times, most recently from a46c7dd to 4279f92 Compare October 24, 2024 23:02
- Make deploy_signed_repos_to_s3 print a message that it is doing the s3sync
- Refactor messy code around it
- Make rspec work
- Update rubocop complaint
@e-gris
Copy link
Collaborator Author

e-gris commented Oct 25, 2024

Verified on platform-ship

Copy link

@nsawlidi-perforce nsawlidi-perforce left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@e-gris e-gris merged commit fb00da1 into puppetlabs:1.0.x Oct 28, 2024
4 checks passed
@e-gris e-gris deleted the deploy_to_s3_feedback branch October 28, 2024 17:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants