Skip to content

Conversation

@sh-rp
Copy link
Collaborator

@sh-rp sh-rp commented Aug 13, 2025

Description

We have a bunch of very useful testing helpers which we would like to share as a separate package to pypi. This PR servers as a starting point to discuss the repository structure with this package. For testing, one single util function is moved to the new package.

Things to decide (with initial suggestions)

  • The name of the test package (pytest-dlt)
  • The versioning and release scheme (1-1 with the main dlt package, we always have the exact same version and release at the same time)
  • License (same as dlt package)
  • Location of package (see current repo, it is in a folder and included in the main workspace via the uv sources section)
  • Folder / module structure within this new test package (no proposals yet)

@netlify
Copy link

netlify bot commented Aug 13, 2025

Deploy Preview for dlt-hub-docs canceled.

Name Link
🔨 Latest commit cf6858b
🔍 Latest deploy log https://app.netlify.com/projects/dlt-hub-docs/deploys/689c99e1908d2300086018f5

@zilto
Copy link
Collaborator

zilto commented Aug 13, 2025

Awesome feature (eager to use it myself). Regarding naming, the convention suggests dlt-pytest instead of pytest-dlt (see results for "pytest" on pypi)

@sh-rp
Copy link
Collaborator Author

sh-rp commented Aug 13, 2025

Awesome feature (eager to use it myself). Regarding naming, the convention suggests dlt-pytest instead of pytest-dlt (see results for "pytest" on pypi)

chat-gpt says the convention is pytest first, but I am not sure that that is true. dlt-pytest sounds nicer to me too.

@zilto
Copy link
Collaborator

zilto commented Aug 13, 2025

I might have been wrong though. This pytest docs page lists a lot of pytest-PROJECT_NAME plugins.

Though, if the number of dlt related projects grow, it can be useful to keep track of them using the dlt- prefix.

I'm ok with either

@sh-rp sh-rp self-assigned this Sep 15, 2025
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.

2 participants