dotnet / dotnet/cli-lab

[.NET Core Uninstall Tool] `below` and `latest` options are inconsistent from a verbiage perspective.

Open
#67 3 comments 0 reactions 0 assignees View on GitHub
Area-Uninstall-Tool design question
Dominant language
C#
Stars
774
Forks
72
Avg merge
1d 23h
Merged PRs (30d)
2

Description

In general we think of versions as a number that increase over time. The appropriate time based comparison terminology are `later` or `earlier`. Terminology such as `below` and `lower` are more appropriate for quantitative numbers. I think this [StackExchange post](https://english.stackexchange.com/a/112263) has a good explanation.

The `.NET Core Uninstall Tool` appears to use a mixture of verbiage - e.g. `below` and `latest`. Consider refactoring the option names to consistently use time based terminology - e.g. `earlier` and `later`.

```
--all-below Remove .NET Core SDKs or Runtimes below the specified version. The specified version
will remain.
--all-but-latest Remove .NET Core SDKs or Runtimes, except the one highest version.
--all-lower-patches Remove .NET Core SDKs or Runtimes that have been superceded by higher patches.
--all-previews-but-latest Remove .NET Core SDKs or Runtimes that are marked as previews, except the one highest
preview.
```

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.