Skip to content

Add dynamic versioning and --version command line option#1048

Merged
cmungall merged 3 commits intolinkml:mainfrom
dalito:issue378
Oct 27, 2022
Merged

Add dynamic versioning and --version command line option#1048
cmungall merged 3 commits intolinkml:mainfrom
dalito:issue378

Conversation

@dalito
Copy link
Copy Markdown
Member

@dalito dalito commented Oct 26, 2022

This addresses partly #1034. Click by default supports only the long format --version but not -V. In my opinion the short form is not important to have and I suggest to stick with click´s default. I added both '-V' and '--version' in a later commit.

Still to do: Adapt all other command line tools. I just added this to linkml-convert as an example to get feedback before continuing.

Closes #378

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Oct 26, 2022

Codecov Report

Merging #1048 (617b825) into main (eb29c53) will decrease coverage by 0.00%.
The diff coverage is 71.42%.

❗ Current head 617b825 differs from pull request most recent head 578bd1d. Consider uploading reports for the commit 578bd1d to get more accurate results

@@            Coverage Diff             @@
##             main    #1048      +/-   ##
==========================================
- Coverage   78.06%   78.06%   -0.01%     
==========================================
  Files          73       74       +1     
  Lines        8390     8398       +8     
  Branches     2063     2063              
==========================================
+ Hits         6550     6556       +6     
- Misses       1446     1448       +2     
  Partials      394      394              
Impacted Files Coverage Δ
linkml/_version.py 60.00% <60.00%> (ø)
linkml/utils/converter.py 51.96% <100.00%> (+1.45%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@cmungall cmungall requested a review from hsolbrig October 26, 2022 14:46
@cmungall
Copy link
Copy Markdown
Member

Thanks so much @dalito 🙏 ! @hsolbrig is trying this out and will write a test for it - I think he has some opinions about -V vs --version too!

Copy link
Copy Markdown
Contributor

@hsolbrig hsolbrig left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! Thanks.

Appreciate the insights on how poetry can get the actual version from github. We should really consider doing that for model versions as well ...

@cmungall
Copy link
Copy Markdown
Member

Good idea @hsolbrig! I made an issue #1051

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.

Add --version flag

4 participants