Skip to content

test(workflows): aligned uv sync command - #150

Merged
dalito merged 1 commit into
linkml:mainfrom
lmodel:sync
May 25, 2026
Merged

test(workflows): aligned uv sync command#150
dalito merged 1 commit into
linkml:mainfrom
lmodel:sync

Conversation

@noelmcloughlin

@noelmcloughlin noelmcloughlin commented Apr 14, 2026

Copy link
Copy Markdown
Contributor

This PR uses just sync --group dev syntax to align with justfile..

uv sync --group dev

@dalito

dalito commented May 24, 2026

Copy link
Copy Markdown
Member

UV adds the dev group by default.

So we could just simplify to uv sync - In current uv (0.11.16) the option --dev that is used in the CI scripts is not documented in the CLI help (but does not error).

...or is the main idea to be consistent and explicit? (I am fine with that)

@dalito dalito left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Making it more explicit that dev is installed is helpful.

@dalito
dalito merged commit e32366a into linkml:main May 25, 2026
10 checks passed
@noelmcloughlin

Copy link
Copy Markdown
Contributor Author

Thanks - I did not analyze too much - thought it might be discrepancy to fix . but its merged now so all good.

@noelmcloughlin
noelmcloughlin deleted the sync branch May 26, 2026 09:42
@dalito dalito added this to the Release 0.5.0 milestone May 27, 2026
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.

2 participants