Skip to content
This repository was archived by the owner on Oct 17, 2025. It is now read-only.

Conversation

@christiannicola
Copy link
Contributor

This PR adds a complete Nix setup to the project. See README.md for more details on how to make use of it.

- it's usually better to just use a predefined uri instead of building
  your own. this patch removes the mongodb parameters and accepts a uri
  now instead.
- remove testcontainers; no longer necessary, we have a working database
  right in our development shell.

CBR-14
this patch changes the docker file to use the exact same setup, using
the nix flake. this ensures that the build / dependencies are exactly
the same.

CBR-14
- the compose stack is no longer needed as we have a working environment
  right in our nix shell
- taskfile has been replaced with a small Makefile

CBR-14
add some basic instructions on how to work with nix

CBR-14
this patch enables the github actions to also use the flake, just like
your local environment. this ensures a reproducable environment, no
matter where the code runs.

CBR-14
@notion-workspace
Copy link

@codecov
Copy link

codecov bot commented Jul 24, 2025

Codecov Report

Attention: Patch coverage is 64.86486% with 13 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
cmd/growteer-api/main.go 0.00% 6 Missing ⚠️
internal/infrastructure/environment/environment.go 0.00% 4 Missing ⚠️
internal/api/server.go 87.50% 1 Missing and 2 partials ⚠️

📢 Thoughts on this report? Let us know!

@christiannicola christiannicola force-pushed the CBR-14 branch 2 times, most recently from 4321dd0 to 8b147a4 Compare July 24, 2025 06:56
Copy link
Contributor

@CBrather CBrather left a comment

Choose a reason for hiding this comment

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

I have to check out nix in more detail later, but it looks very solid as long as we are careful not to run the env anywhere non-local (due to the secrets logged). So I'll go with the flow.

@christiannicola christiannicola enabled auto-merge (squash) July 24, 2025 09:18
@christiannicola
Copy link
Contributor Author

christiannicola commented Jul 24, 2025

I have to check out nix in more detail later, but it looks very solid as long as we are careful not to run the env anywhere non-local (due to the secrets logged). So I'll go with the flow.

nix shell is a development environment and not meant for actually deploying the application. The current Helm chart is still the indended way of deploying it.

There are a lot of options that are open to us now in regards to the project setup:

@christiannicola christiannicola merged commit 35b9b9e into growteer:main Jul 24, 2025
2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants