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

Remove archived dependency and add --version #100

Merged
merged 2 commits into from
Nov 6, 2024
Merged

Conversation

hugopl
Copy link
Contributor

@hugopl hugopl commented Nov 6, 2024

Hi,

hugopl/version_from_shard was archived by me, so I'm replacing it by a simple macro.

It get the version from shards.yml and add a build to it with a short git hash or "unknown" if it wasn't built from a git repository (e.g. tarballs).

So version 0.15.0 is now show as 0.15.0+f8aa5c6, i.e. the build part of the version will always present. If this isn't desired I can update the PR.

Then I also add a --version flag, so neovim (and probably other editors) can show crystalline version on UI, nowadays it can't:

image

With this patch:

image

This shard was archived by me, the macro above does almost the same.

It get the version from shards.yml and add a build to it with a short
git hash or "unknown" if it wasn't built from a git repository (e.g. tarballs).

So version 0.15.0 is now show as 0.15.0+f8aa5c6.
@elbywan
Copy link
Owner

elbywan commented Nov 6, 2024

Hey @hugopl 👋

hugopl/version_from_shard was archived by me, so I'm replacing it by a simple macro.

Oh, thanks for dedicating some time to update this! 🙇

@elbywan elbywan merged commit bfb0b3b into elbywan:master Nov 6, 2024
4 checks passed
@zw963
Copy link

zw963 commented Nov 7, 2024

Thanks, crstalline have --version now finally.

@hugopl hugopl deleted the version branch November 10, 2024 04:57
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.

3 participants