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

Update kinobi version #12

Merged
merged 14 commits into from
Mar 5, 2024
Merged

Update kinobi version #12

merged 14 commits into from
Mar 5, 2024

Conversation

febo
Copy link
Contributor

@febo febo commented Mar 4, 2024

No description provided.

@febo febo marked this pull request as ready for review March 4, 2024 22:39
.github/workflows/build-cli.yml Outdated Show resolved Hide resolved
clients/rust/asset/Cargo.toml Show resolved Hide resolved
@samuelvanderwaal
Copy link
Contributor

I'm a little concerned about the CLI actually being in a workspace with the other crates but we can try it and see how it works.

I scanned the Kinobi changes but am not really in a position to properly check them. Everything seems to build and generate fine in a brand new repo tho, so lgtm.

The new update typescript helper is a great improvement in usability!

@febo
Copy link
Contributor Author

febo commented Mar 5, 2024

I noticed that this structure is both faster to build and uses less space since the dependencies get "shared" across the different crates.

@febo febo merged commit 9eec4d9 into main Mar 5, 2024
@febo febo deleted the febo/dependencies-bump branch March 5, 2024 07:50
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