Skip to content

Question about third party library naming guidelines #4811

Answered by kiview
nagyesta asked this question in Q&A
Discussion options

You must be logged in to vote

Hi @nagyesta, thanks a lot for contributing to the Testcontainers ecosystem! 🙌
For our own artifacts, since they are part of the Testcontainers build, we don't add an additional testcontainers suffix, but in think in your case, this is totally fine for now.

The class name of the container extending GenericContainer should be something like ${technology}Container, so your name of LowkeyVaultContainer is well aligned.

Something we like to enforce for our own modules (and we would suggest you to consider) is to have the API only use standard types. In your case, returning TokenCredential would violate this guideline. The better maintainable approach might be, to just expose username and pass…

Replies: 1 comment 1 reply

Comment options

You must be logged in to vote
1 reply
@nagyesta
Comment options

Answer selected by nagyesta
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Category
Q&A
Labels
None yet
2 participants