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

convert-examples-repo-to-defs cloud nux quickstart_aws #11135

Conversation

yuhan
Copy link
Contributor

@yuhan yuhan commented Dec 14, 2022

Summary & Motivation

convert quickstart_aws example after it's moved from a standalone repo back to the mono repo (#11130)

  • workspace.yaml -> pyproject.toml
  • @repository -> Definitions
  • repository.py -> __init__.py

How I Tested These Changes

  • local dagit loads
  • unit

@vercel
Copy link

vercel bot commented Dec 14, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Ignored Deployments
Name Status Preview Comments Updated
dagit-storybook ⬜️ Ignored (Inspect) Dec 15, 2022 at 2:16AM (UTC)
dagster ⬜️ Ignored (Inspect) Dec 15, 2022 at 2:16AM (UTC)

@yuhan yuhan force-pushed the yuhan/12-14-move_quickstart_snowflake_back_to_mono_repo branch from 77fc49b to 484d0df Compare December 15, 2022 01:47
@yuhan yuhan force-pushed the yuhan/12-14-convert-examples-repo-to-defs_cloud_nux_quickstart_aws branch from 6a45bac to e13fcd0 Compare December 15, 2022 01:47
@yuhan yuhan requested review from schrockn and sryza December 15, 2022 01:51
Base automatically changed from yuhan/12-14-move_quickstart_snowflake_back_to_mono_repo to master December 15, 2022 02:11
@yuhan yuhan force-pushed the yuhan/12-14-convert-examples-repo-to-defs_cloud_nux_quickstart_aws branch from e13fcd0 to 064c730 Compare December 15, 2022 02:16
@yuhan yuhan merged commit 5fdecbd into master Dec 15, 2022
@yuhan yuhan deleted the yuhan/12-14-convert-examples-repo-to-defs_cloud_nux_quickstart_aws branch December 15, 2022 04:32
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