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

[components] project_name -> code_location_name in scaffolded pyproject.toml #27480

Merged

Conversation

smackesey
Copy link
Collaborator

@smackesey smackesey commented Feb 1, 2025

Summary & Motivation

The tool.dagster section in our scaffolded pyproject.toml contained a project_name field. However, this isn't actually read by our tooling. What is read is a code_location_name field. This renames the project_name field to code_location_name. It affects both dagster and dagster-dg (which had started by copying the pyproject.toml scaffolding from dagster core).

How I Tested These Changes

Tweaked unit tests.

Copy link
Collaborator Author

smackesey commented Feb 1, 2025

@smackesey smackesey changed the base branch from sean/components/code-location-scaffold-workspace to graphite-base/27480 February 1, 2025 16:18
@smackesey smackesey force-pushed the graphite-base/27480 branch from 42436ff to 94e2d67 Compare February 1, 2025 16:18
@smackesey smackesey force-pushed the sean/components/fix-code-location-name-pyproject-toml branch from cc1a95e to 59253a3 Compare February 1, 2025 16:18
@smackesey smackesey changed the base branch from graphite-base/27480 to master February 1, 2025 16:18
@smackesey smackesey force-pushed the sean/components/fix-code-location-name-pyproject-toml branch from 59253a3 to 47f927a Compare February 1, 2025 16:23
@smackesey smackesey force-pushed the sean/components/fix-code-location-name-pyproject-toml branch from 47f927a to 2f17209 Compare February 1, 2025 16:31
@smackesey smackesey marked this pull request as ready for review February 1, 2025 16:40
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