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

Update to .NET 5.0, add option for pruning #17

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

TYoungSL
Copy link

@TYoungSL TYoungSL commented Jun 22, 2021

Updates the project to .NET 5.0. Updates package dependencies.

Adds an option (-p, --prune) for pruning of versions other than latest release and prerelease.

Changes default for last accessed minimum days to 90.

Compares time in UTC.

Adds redundant fallback for LastAccessTimeUtc to LastWriteTimeUtc.

Properly collects old versions of tools (.tools subdirectory) instead of wiping the entire toolset.

Reports some non-trivial directory deletion failure conditions.

Note: Fallback to using LastWriteTimeUtc is probably over-redundant paranoia.

The dependency on NuGet.Versioning means it would not be hard to add functionality to keep N previous releases, keep N prereleases, keep latest of major versions, etc.

TYoungSL added 3 commits June 22, 2021 16:45
bump deps

set default to 90 days

handle no exceptions from missing last access time, fall back to last write

report some exceptions during clean up

bump version
…eases other than the latest

add dependency on NuGet.Versioning to parse folder versions and skip extraneous directories

properly handle tools packages sub-directory
unify deletion exception handling, sizing, version checking
@TYoungSL
Copy link
Author

TYoungSL commented Jun 22, 2021

Addresses issue #16

Might address issue #11

@martinussuherman
Copy link

@TYoungSL could you fork this repo and integrate this PR on your fork? Probably also update to .NET 6?
This repo doesn't seem to be maintained anymore imo.

Regards,

Martinus

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