Skip to content

Conversation

@bjzsh
Copy link
Member

@bjzsh bjzsh commented Sep 9, 2025

Not the most clean in terms of error handling and arg parsing, but it will work fine. It's basically a script so I think it doesn't need to be too polished until we we need to add more things to do.

@bjzsh bjzsh requested review from Jaden51 and joshhwuu September 9, 2025 07:12
@bjzsh bjzsh self-assigned this Sep 9, 2025
@bjzsh bjzsh added the enhancement New feature or request label Sep 9, 2025
Copy link
Contributor

@Jaden51 Jaden51 left a comment

Choose a reason for hiding this comment

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

What are the .env vars to run the seed?

Thanks for the refactors on keygeneration.go! Do the tests still pass?

@bjzsh
Copy link
Member Author

bjzsh commented Sep 10, 2025

What are the .env vars to run the seed?

Just EC_DSN for the database connection. Same as the application.

Thanks for the refactors on keygeneration.go! Do the tests still pass?

Yes.

@bjzsh bjzsh requested a review from Jaden51 September 10, 2025 02:46
Copy link
Contributor

@Jaden51 Jaden51 left a comment

Choose a reason for hiding this comment

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

Thanks Brian 🚢 !

@bjzsh bjzsh merged commit 001349e into main Sep 10, 2025
1 check passed
@bjzsh bjzsh deleted the feat/db-seeder branch September 10, 2025 20:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

No way to generate an API Key without an API Key

3 participants