Skip to content

Conversation

@nyonson
Copy link
Contributor

@nyonson nyonson commented Oct 29, 2025

Draft because this is a bit of a stretch, but the quick and dirty way to get the lock file script here. I think this is the limit of the "ci" task runner (abusing it here), but would hold off breaking it up until attempting to move the API tooling.

@tcharding
Copy link
Member

I like the idea of moving the update logic to this repo but I'm not sure about putting it in the CI script because its not something we would ever run in CI, right? Why not just put a new shell script here?

@tcharding
Copy link
Member

Is this PR just the last patch or the first 7 too? Not sure where we are up to with the quiet flag. Everything else looks good.

This is probably stretching the limits of the CI run task
script, but it leverages a lot of the internal tooling.
@nyonson
Copy link
Contributor Author

nyonson commented Nov 5, 2025

I like the idea of moving the update logic to this repo but I'm not sure about putting it in the CI script because its not something we would ever run in CI, right? Why not just put a new shell script here?

Yea, this is purely laziness on my part. The task leverages tools which are only in run_task atm, need_toolchain and verbose_say. I can work on that "stdlib" vision you had for these shared functions first if you think its worth it. Definitely will be if we want to move the API check stuff down here too.

Is this PR just the last patch or the first 7 too?

I rebased it to just the single commit, but it needs the toolchain functionality in #23

@tcharding
Copy link
Member

The task leverages tools which are only in run_task atm, need_toolchain and verbose_say.

I'd just cut n paste, that is what I usually do with these 'bash stdlib' functions. Although need_toolchain is a bit more complex now so maybe that is too much code to copy around? Up to you. There is only so much time in a day and messing with shell too much is a time sync IMO.

@tcharding
Copy link
Member

tcharding commented Nov 5, 2025

FYI we have an idea to use xshell to replace all the shell but no one has gotten around to it yet. The counter argument is that LLMs are pretty good at bash.

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