-
Notifications
You must be signed in to change notification settings - Fork 3
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
Merge in "box packer" script #30
Comments
The script is currently here: https://gist.github.com/kierdavis/ddd6da1c76119841264f20cdaee1958d Note that this isn't a matter of downloading and committing the file - it really needs a lot of polish and should talk directly to the rest of this package to operate the inventory rather than calling out to subcommands. I'd recommend the script be used as a proof-of-concept/reference implementation for creating a new |
We used this again for SR2022. |
One thing which would be good to do before merging into the shared tools would be to separate out the specification for what should go in the box from the implementing code. Otherwise we end up with tooling that's tied to a specific year's kit. |
So I don't think we can put that in the inventory repo yet? For the same reason as we can't add a README? We could bake a definition file into this package but that doesn't feel right. |
I think there's some space in the |
@kierdavis wrote a box packing script for kit packing. We should merge it in here.
The text was updated successfully, but these errors were encountered: