Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

updated save flag and related logs #404

Merged
merged 2 commits into from
Feb 5, 2025
Merged

updated save flag and related logs #404

merged 2 commits into from
Feb 5, 2025

Conversation

zackbradys
Copy link
Member

@zackbradys zackbradys commented Feb 5, 2025 •
edited
Loading

Please check below, if the PR fulfills these requirements:

  • Commit(s) and code follow the repositories guidelines.
  • Test(s) have been added or updated to support these change(s).
  • Doc(s) have been added or updated to support these change(s).

Associated Links:

  • N/A

Types of Changes:

  • Feature

Proposed Changes:

  • Cleaned up code associated with hauler store save
  • Updated the default haul (archive) name to haul.tar.zst

Verification/Testing of Changes:

  • Passed all unit and integration tests...

Additional Context:

  • N/A

@zackbradys zackbradys self-assigned this Feb 5, 2025
@zackbradys zackbradys added enhancement New feature or request size/M Denotes an issue/PR requiring a relatively moderate amount of work labels Feb 5, 2025
@zackbradys zackbradys added this to the Hauler v1.2.0 milestone Feb 5, 2025
@zackbradys zackbradys requested a review from a team February 5, 2025 07:24
amartin120
amartin120 previously approved these changes Feb 5, 2025
View reviewed changes
Copy link
Contributor

@amartin120 amartin120 left a comment

Choose a reason for hiding this comment

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

LGTM

@zackbradys
Merge branch 'main' into save-flag-updates
6828694 
Signed-off-by: Zack Brady <zackbrady123@gmail.com>
@zackbradys zackbradys merged commit 535a82c into hauler-dev:main Feb 5, 2025
1 check passed
@zackbradys zackbradys deleted the save-flag-updates branch February 5, 2025 14:39
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@amartin120 amartin120 amartin120 approved these changes

Assignees

@zackbradys zackbradys

Labels
enhancement New feature or request size/M Denotes an issue/PR requiring a relatively moderate amount of work
Projects
Hauler
Status: Resolved
Milestone
Hauler v1.2.0
Development

Successfully merging this pull request may close these issues.
2 participants
@zackbradys @amartin120