Skip to content

refactor justfiles / test commands - #3459

Merged
syphar merged 3 commits into
rust-lang:mainfrom
syphar:just-local-cli
Aug 31, 2026
Merged

refactor justfiles / test commands #3459
syphar merged 3 commits into
rust-lang:mainfrom
syphar:just-local-cli

Conversation

@syphar

@syphar syphar commented Aug 18, 2026

Copy link
Copy Markdown
Member

I tried to revisit all the just commands, coming from my current usage under linux, also thinking about my past usage as mac user. #3474, #3472, #3475, and #3473 were actually intermediate steps I pulled out into separate PRs.

My goal is actually that I'm using all these commands myself, instead of my shell history.

Main points:

  • by default on a dev machine, the binaries should be directly used, locally via cargo run -p, on all operating systems. You can switch to using the containers if you want.
  • the builder should do cargo run only on linux, otherwise use the builder docker container, since our builder doesn't work otherwise.
  • gui-tests should run without docker on dev machines, only in CI we want all in docker to know it works. In any case, I dropped the Dockerfile.gui-tests and we'll always run the tests on the runner. With this, locally iterating on the gui-tests works super smooth.
  • I want to easily run the CI checks locally via a recipe, when fixing stuff.

Generally I see this just as "the first idea", and will iterate over the next weeks when I use them.

Other small changes:

  • all these super small cli-xx- recipes are dropped, people can just use just cli SOME ARGS. Downside: no recipe autocomplete. Not sure yet what I like more.

@syphar syphar self-assigned this Aug 18, 2026
@github-actions github-actions Bot added the S-waiting-on-review Status: This pull request has been implemented and needs to be reviewed label Aug 18, 2026
@rustbot

This comment has been minimized.

@rustbot

This comment has been minimized.

@syphar syphar changed the title WIP: update justfiles so they can run stuff locally on linux refactor justfiles / test commands Aug 26, 2026
@syphar

This comment was marked as off-topic.

@syphar
syphar marked this pull request as ready for review August 26, 2026 17:48
@syphar
syphar requested a review from a team as a code owner August 26, 2026 17:48
@syphar

syphar commented Aug 26, 2026

Copy link
Copy Markdown
Member Author

r? @GuillaumeGomez

@rustbot rustbot assigned GuillaumeGomez and unassigned syphar Aug 26, 2026
Comment thread package-lock.json Outdated
@syphar
syphar merged commit f53386c into rust-lang:main Aug 31, 2026
7 checks passed
@syphar
syphar deleted the just-local-cli branch August 31, 2026 10:38
@github-actions github-actions Bot added S-waiting-on-deploy This PR is ready to be merged, but is waiting for an admin to have time to deploy it and removed S-waiting-on-review Status: This pull request has been implemented and needs to be reviewed labels Aug 31, 2026
@syphar syphar removed the S-waiting-on-deploy This PR is ready to be merged, but is waiting for an admin to have time to deploy it label Sep 1, 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.

3 participants