Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[components] Move dg cli back to verb-first (rearrange tests) #27921

Merged
merged 1 commit into from
Feb 20, 2025

Conversation

smackesey
Copy link
Collaborator

@smackesey smackesey commented Feb 19, 2025

Summary & Motivation

Update test organization of dagster-dg to reflect new verb-first orientation.

How I Tested These Changes

Unit tests

Copy link
Collaborator Author

smackesey commented Feb 19, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@smackesey smackesey force-pushed the sean/components/move-cli-to-verb-first-tests branch 2 times, most recently from 670c685 to 75eb940 Compare February 19, 2025 21:32
@smackesey smackesey force-pushed the sean/components/move-cli-to-verb-first branch 2 times, most recently from 99aadf6 to bfe40f8 Compare February 19, 2025 22:03
@smackesey smackesey force-pushed the sean/components/move-cli-to-verb-first-tests branch from 75eb940 to 7442e60 Compare February 19, 2025 22:03
@smackesey smackesey marked this pull request as ready for review February 19, 2025 22:04
@smackesey smackesey force-pushed the sean/components/move-cli-to-verb-first branch from bfe40f8 to 77b5d19 Compare February 19, 2025 22:15
@smackesey smackesey force-pushed the sean/components/move-cli-to-verb-first-tests branch from 7442e60 to f9be519 Compare February 19, 2025 22:15
Copy link
Member

@schrockn schrockn left a comment

Choose a reason for hiding this comment

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

didntreadlol

@smackesey smackesey force-pushed the sean/components/move-cli-to-verb-first-tests branch from f9be519 to ac00deb Compare February 20, 2025 10:11
@smackesey smackesey force-pushed the sean/components/move-cli-to-verb-first branch from 77b5d19 to 397a414 Compare February 20, 2025 10:11
@smackesey smackesey force-pushed the sean/components/move-cli-to-verb-first-tests branch from ac00deb to d67b573 Compare February 20, 2025 19:18
@smackesey smackesey force-pushed the sean/components/move-cli-to-verb-first branch from 397a414 to b5700e5 Compare February 20, 2025 19:18
Base automatically changed from sean/components/move-cli-to-verb-first to master February 20, 2025 19:40
@smackesey smackesey requested a review from neverett as a code owner February 20, 2025 19:40
@smackesey smackesey force-pushed the sean/components/move-cli-to-verb-first-tests branch from d67b573 to 42ab9f5 Compare February 20, 2025 19:41
@smackesey smackesey merged commit cbb54ce into master Feb 20, 2025
3 of 4 checks passed
@smackesey smackesey deleted the sean/components/move-cli-to-verb-first-tests branch February 20, 2025 19:41
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants