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

Support --stdin-quit functionality #217

Merged
merged 4 commits into from Jan 11, 2023
Merged

Conversation

michaeljones
Copy link

The core behaviour has been added to the watchexec lib so we update to the latest watchexec and add the flag & hook into the runtime to get the desired behaviour.

This follows the same pattern as the implementation for the watchexec binary. I hope it is satisfactory.

No rush on this. I realise it involves releasing watchexec and updating this reference which may or may not be something you plan to do soon.

Checklist

  • Switch to explicit version of watchexec when published (rather than a git sha ref)

The core behaviour has been added to the watchexec lib so we update to
the latest watchexec and add the flag & hook into the runtime to get
the desired behaviour.

The switch to a git sha rev for watchexec should be temporary.

The other package updates seemed to be required and come from deleting
the Cargo.lock entries for once_cell and thiserror to allow them to
update and then updating the version of miette to match the one used
by watchexec.
@passcod
Copy link
Member

passcod commented Dec 2, 2022

Yeah, I've got a few things I want to do before I release a cargo watch with the new watchexec engine, but I plan to get to that during the summer (december here) break! So it shouldn't be too long :)

@passcod passcod enabled auto-merge (squash) January 11, 2023 13:37
@passcod passcod merged commit 17c63c9 into watchexec:main Jan 11, 2023
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.

None yet

2 participants