search

XcodesOrg / xcodes

The best command-line tool to install and switch between multiple versions of Xcode.
MIT License
3.74k stars 130 forks source link

use simpler 'release' instead of 'non-prerelease' word in user-facing… #239

Closed pastey closed 1 year ago

pastey commented 1 year ago

… output

Hi,

Not sure if you'll find this PR useful and corresponding to your philosophy, but anyway...

I find current phrasing Latest non-prerelease version available is... a bit too complex. I would suggest to simplify it by replacing non-prerelease to plain... release.

I see that the same NonPrerelease term is used in variable names. If you find this suggestion feasible, then I can rename variables too.

Maybe you'll tell me that there's some deep idea behind non-prerelease, then I'm fine with that too :)

Thanks for the great tool that saves ton of my time!

pastey commented 1 year ago

@MattKiazyk, please check 🙏

MattKiazyk commented 1 year ago

Sorry @pastey can you rebase with the recent changes with runtime installs. Looks good otherwise.

pastey commented 1 year ago

Sorry @pastey can you rebase with the recent changes with runtime installs. Looks good otherwise.

Done. Hope I did it correctly (I'm more used to plain old merge)

pastey commented 1 year ago

Hey, @MattKiazyk will you have a chance to take one more look at this PR?

pastey commented 1 year ago

Hey, @MattKiazyk will you have a chance to take one more look at this PR?

MattKiazyk commented 1 year ago

Thanks for the change!