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

feat: Adds flake #517

Merged
merged 4 commits into from
Dec 17, 2024
Merged

Conversation

thomastaylor312
Copy link
Contributor

This adds a flake.nix and associated files for a dev environment via nix and steps for building all the parts of wadm in an incremental way. Part of this was an experiment to see what we could get nix to do with monorepo deps in the simple case. Please note that this does not actually add this as part of the build pipeline, just adds the file the repo for people to use. Maybe in the future we can add in builds and caching into our pipelines.

thomastaylor312 and others added 3 commits December 13, 2024 17:21
Adds a nix flake for usage in building things. It is still missing the
ability to run an e2e test and build docker images, but it does work
for both building and nix shell

Signed-off-by: Taylor Thomas <[email protected]>
Signed-off-by: Roman Volosatovs <[email protected]>
@thomastaylor312 thomastaylor312 requested a review from a team as a code owner December 14, 2024 00:34
@thomastaylor312
Copy link
Contributor Author

Special thanks to @rvolosatovs for helping me figure out the cargo.toml munging

serde_json::to_vec(&DeleteModelRequest {
version: None,
}).unwrap(),
serde_json::to_vec(&DeleteModelRequest { version: None }).unwrap(),
Copy link
Contributor Author

@thomastaylor312 thomastaylor312 Dec 14, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note: These changes were just a missing cargo fmt that I found while running tests in the flake

Copy link
Member

@brooksmtownsend brooksmtownsend left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All looks good to me, only a supernit and a question. Is it worth throwing a comment at the top of the flake marking it as experimental? Only to consider whether or not we're going to update & maintain it often

@thomastaylor312 thomastaylor312 enabled auto-merge (rebase) December 17, 2024 19:25
@thomastaylor312 thomastaylor312 merged commit 38cb50f into wasmCloud:main Dec 17, 2024
4 of 5 checks passed
@thomastaylor312 thomastaylor312 deleted the exp/use_nix branch December 17, 2024 20:55
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.

3 participants