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

chore: devenv environment #2083

Open
wants to merge 1 commit into
base: devel
Choose a base branch
from

Conversation

joscha
Copy link
Contributor

@joscha joscha commented Nov 21, 2024

Description

This adds a Devenv environment to the repository which allows immediate development w/o have to worry about local installation of dependencies (make, python, poetry, etc.).
Devenv is using nix under the hood, which provides a reproducible environment due to the fact that it controls the whole dependency closure. That means that the files in this PR would ensure that all developers would use the same version of make, python, poetry, etc. for their local development of dlt, which reduces a whole bunch of problems when talking about issues, opening pull requests, etc.

If the system has direnv and devenv installed everything is automatic.
After activating the environment, setup can be run, which calls out to make dev.

Also happy to swap this for a raw nix flake or even a nix-shell if devenv is too bleeding edge.

Copy link

netlify bot commented Nov 21, 2024

Deploy Preview for dlt-hub-docs canceled.

Name Link
🔨 Latest commit 9141ee4
🔍 Latest deploy log https://app.netlify.com/sites/dlt-hub-docs/deploys/673f1e84440fe800097c7c89

'';

# https://devenv.sh/pre-commit-hooks/
git-hooks.hooks.lint = {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this runs the linter before commit. Can be omitted by passing -n, however I am also happy to remove this, if in question.

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.

1 participant