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

modified update notification #58

Merged
merged 2 commits into from
Jan 24, 2024
Merged

modified update notification #58

merged 2 commits into from
Jan 24, 2024

Conversation

thomas-mc-work
Copy link
Contributor

Slightly changed the notation of the local/latest version to simplify the visual impression. But it's a matter of taste.

Slightly changed the notation of the local/latest version to simplify the visual impression. But it's a matter of taste.
@mag37
Copy link
Owner

mag37 commented Jan 24, 2024

Looking good!
Do you know if the arrow works no matter the font used?

@thomas-mc-work
Copy link
Contributor Author

Thank you!

I'm not 100 % sure, but I've noticed that the Unicode adoption is pretty well since some years. And I find those glyphs more and more i nshell scripts.

@mag37
Copy link
Owner

mag37 commented Jan 24, 2024

Just tried it in the 5 different machines I got access to right now (4 different distros, 3 different prompts/shells) and it worked fine in all.
I'll merge it.

Thank you for the contribution!

@mag37 mag37 merged commit ea096b5 into mag37:main Jan 24, 2024
@thomas-mc-work thomas-mc-work deleted the patch-1 branch January 24, 2024 22:10
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.

None yet

2 participants