Migrate scripts to xtask pattern #1096
simonbuchan
started this conversation in
Ideas
Replies: 3 comments
-
I think it's a good idea |
Beta Was this translation helpful? Give feedback.
0 replies
-
Neat! The main ones to start with are probably the ones in https://stc.dudy.dev/docs/contributing - I'm thinking starting with something like:
I'm happy to iterate on workflow if you have any ideas, though! |
Beta Was this translation helpful? Give feedback.
0 replies
-
Opened a draft in #1097 but seems I can't mark PRs as draft :( There's a few question mark details there about what variables should be set, etc., but the API above is basically there.
|
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
As a filthy Windows user, the current scripts setup for tests etc., is a bit fiddly to work with.
I have a branch that has an initial stab at adding the xtask pattern and migrating
crates/stc_ts_type_checker/scripts/check.sh
to it, which seems to have worked out pretty well - the code is quite straightforward and the only real downside is it's a crate that has to be built the first time, andCommand
is a bit uglier to work with directly than shell. On the other side, the Rust to filter, trim and sort the output of the conformance tests pass list is quite simple, in comparison to shell.Check out the branch here: main...simonbuchan:stc:xtask
For reference, the xtask pattern is essentially the closest equivalent to npm scripts for cargo: see this blog post for a description; but the TDLR is you can just clone and run
cargo xtask check
on any platform instead ofcrates/stc_ts_type_checker/scripts/check.sh
on linux / mac.Is this something you would be interested in?
Beta Was this translation helpful? Give feedback.
All reactions