Skip to content

Feature: sort#13

Open
Timple wants to merge 3 commits into
RoboStack:mainfrom
nobleo:feature/sort
Open

Feature: sort#13
Timple wants to merge 3 commits into
RoboStack:mainfrom
nobleo:feature/sort

Conversation

@Timple

@Timple Timple commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@Timple Timple changed the title Feature/sort Feature: sort Jul 8, 2026
@Tobias-Fischer

Copy link
Copy Markdown
Contributor

Does this work on windows?

@Timple

Timple commented Jul 8, 2026

Copy link
Copy Markdown
Contributor Author

It's only python, so that would be my guess..

And at the moment it's not enforced 🙃

Comment thread pixi.toml Outdated
create_snapshot = { cmd = "vinca-snapshot -d lyrical -o rosdistro_snapshot.yaml" }
upload = "rattler-build upload prefix -c robostack-lyrical --generate-attestation"
build_continue_on_failure = { cmd = "rattler-build build --recipe-dir ./recipes -m ./conda_build_config.yaml -c robostack-lyrical -c https://repo.prefix.dev/conda-forge --continue-on-failure --skip-existing", depends-on = ["generate-recipes"] }
sort = "sh -lc 'vinca-sort-vinca-lists $@ vinca.yaml && vinca-sort-yaml-keys $@ pkg_additional_info.yaml robostack.yaml rosdistro_additional_recipes.yaml' --"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I meant this (to me) odd looking syntax

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ah, good catch!

Any suggestions here?
Could make 2 tasks and a 3th depending on the other two?

I was merely trying to pass --check to both sort functions

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yeah separate pixi tasks with shared arguments?

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