Page MenuHomePhabricator

Rename repository.yaml to dagster.yaml
AbandonedPublic

Authored by sashank on Mon, Sep 16, 10:03 PM.

Details

Reviewers
schrockn
Group Reviewers
Restricted Project
Summary

Rename repository.yaml to dagster.yaml.

Changes made:

  • change all existing repository.yaml files to dagster.yaml, repository_file.yaml files to dagster_file.yaml, and *_repository.yaml files to *_dagster.yaml
  • rename cli flags from --repository-yaml to --dagster-yaml
  • change references to repository.yaml in documentation
  • rename ExecutionTargetHandle.for_repo_yaml method to ExecutionTargetHandle.for_dagster_yaml and _ExecutionTargetHandleData attribute repository_yaml to dagster_yaml
Test Plan

unit

Diff Detail

Repository
R1 dagster
Branch
rename-repository-yaml
Lint
Lint OK
Unit
No Unit Test Coverage

Event Timeline

sashank created this revision.Mon, Sep 16, 10:03 PM
sashank updated this revision to Diff 4772.Mon, Sep 16, 10:27 PM

formatting

sashank edited the summary of this revision. (Show Details)Mon, Sep 16, 10:31 PM
sashank added a reviewer: Restricted Project.
sashank updated this revision to Diff 4777.Mon, Sep 16, 11:25 PM

rerun tests

πŸ€”

this breaking change is pretty likely to effect current users - we should think about how we want to handle that

would like to get more input from the team on what we should do here

schrockn requested changes to this revision.Wed, Sep 25, 5:30 PM
schrockn added a subscriber: schrockn.

Per discussion I believe we decided to punt on this, until we get to a multi-repo world where this is more flagrantly needed. Right now we don't know enough about where this will end up and we don't want to thrash our users twice.

This revision now requires changes to proceed.Wed, Sep 25, 5:30 PM
sashank abandoned this revision.Wed, Sep 25, 8:04 PM