dagster-io / dagster

An orchestration platform for the development, production, and observation of data assets.
https://dagster.io
Apache License 2.0
11.79k stars 1.48k forks source link

Avoid unnecessary pyenv usage in environment setup #26090

Open deepyaman opened 21 hours ago

deepyaman commented 21 hours ago

Summary & Motivation

Resolve #26078

How I Tested These Changes

The change in instructions reflects how I set up my local environment. I haven't tested the Buildkite part yet.

Changelog

Avoided unnecessary pyenv usage in environment setup step of contributing guide.

github-actions[bot] commented 21 hours ago

Deploy preview for dagster-docs ready!

Preview available at https://dagster-docs-9ya20d7wz-elementl.vercel.app https://deepyaman-docs-use-uv-venv.dagster.dagster-docs.io

Direct link to changed pages:

neverett commented 20 hours ago

Tagging @alangenfeld and @smackesey since this is a change to the core engineering development process and not strictly a docs change.

deepyaman commented 20 hours ago

Hmm... this seems to pass by and large (helm failure seems unrelated), but I also haven't figured out yet where changes to python_modules/automation/automation/docker/images/buildkite-test/Dockerfile are actually getting used. The actual tests seem to be getting run via tox config generated in .buildkite/dagster-buildkite/dagster_buildkite/steps/tox.py.