Skip to content

Add timestamp to snapshot file names #1344

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

Merged
merged 2 commits into from
Apr 29, 2024

Conversation

bennettgoble
Copy link
Member

@bennettgoble bennettgoble commented Apr 27, 2024

This changeset adds a timestamp in the format of "YYYY-MM-DD_HMS" to snapshot filenames. This is useful for understanding when a snapshot was taken, chronologically ordering files, and is less confusing than the current method of adding a number to the snapshot name, as it does not result in interleaving of old and new snapshots inside a directory.

Closes #1343

This was a broken test that got all the way to viewer release and the main
branch.

(cherry picked from commit a33a9d2)
This changeset adds a timestamp in the format of "YYYY-MM-DD_HHSS" to
snapshot filenames. This is useful for understanding when a snapshot was
taken, chronologically ordering files, and is less confusing than the
current method of adding a number to the snapshot name, as it does not
result in interleaving of old and new snapshots inside a directory.
@bennettgoble bennettgoble force-pushed the signal/snapshot-time branch from 16a1676 to 204b7ff Compare April 27, 2024 18:02
@bennettgoble bennettgoble added the enhancement New feature or request label Apr 27, 2024
@bennettgoble bennettgoble requested a review from kylelinden April 29, 2024 16:05
Copy link

@kylelinden kylelinden left a comment

Choose a reason for hiding this comment

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

Approved. Obvious quality of life improvement.

Copy link
Contributor

@marchcat marchcat left a comment

Choose a reason for hiding this comment

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

How about merging this into Maint C?

@bennettgoble bennettgoble changed the base branch from main to release/maint-c April 29, 2024 23:29
@bennettgoble bennettgoble merged commit c05c066 into release/maint-c Apr 29, 2024
12 checks passed
@bennettgoble bennettgoble deleted the signal/snapshot-time branch April 29, 2024 23:29
@bennettgoble
Copy link
Member Author

@marchcat merged into maint-c, thanks.

@github-actions github-actions bot locked and limited conversation to collaborators Apr 29, 2024
@bennettgoble bennettgoble added this to the Maintenance C milestone May 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
c/cpp enhancement New feature or request llcommon
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Snapshots: Timestamp in Filename
6 participants