Skip to content

Comments

Update format_commands CLI invocation and revert dbt test changes#205

Merged
cnolanminich merged 2 commits intomasterfrom
copilot/update-dagster-cloud-cli-logic
Feb 21, 2026
Merged

Update format_commands CLI invocation and revert dbt test changes#205
cnolanminich merged 2 commits intomasterfrom
copilot/update-dagster-cloud-cli-logic

Conversation

Copy link
Contributor

Copilot AI commented Feb 21, 2026

Updates the dagster-cloud job launch command format in the changed assets script and reverts temporary dbt model changes that were added for testing.

Script: scripts/get_changed_assets.py

  • format_commands now emits asset keys using the key:"..." selector syntax and drops the --repository flag:
# Before
asset_args = " ".join(f"--asset-key {key}" for key in asset_keys)
# --location {location} --repository {repository} --job {job} {asset_args}

# After
asset_args = " ".join(f"--asset-key 'key:\"{key}\"'" for key in asset_keys)
# --location {location} --job {job} {asset_args}

DBT changes reverted

  • Deleted dbt_project/models/CLEANED/orders_aggregated.sql
  • orders_cleaned.sql: removed dt as dt_2 column
  • component.yaml: removed orders_aggregated from the first group's selection
Original prompt

Can you update the scripts/ materialize chaned assets to use this logic for the dagster-cloud CLI invocation?

def format_commands(
grouped: dict[tuple[str, str], list[str]],
job: str,
) -> str:
lines = []
for (location, repository), asset_keys in sorted(grouped.items()):
asset_args = " ".join(f"--asset-key 'key:"{key}"'" for key in asset_keys)
lines.append(
f"dagster-cloud job launch"
f" --location {location}"
f" --job {job}"
f" {asset_args}"
)
return "\n".join(lines)

And then remove the dbt changes I made to orders cleaned, orders aggregated, and the component?


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: cnolanminich <12430096+cnolanminich@users.noreply.github.com>
Copilot AI changed the title [WIP] Update scripts to format dagster-cloud CLI invocation Update format_commands CLI invocation and revert dbt test changes Feb 21, 2026
Copilot AI requested a review from cnolanminich February 21, 2026 00:43
@cnolanminich cnolanminich marked this pull request as ready for review February 21, 2026 00:45
@cnolanminich cnolanminich merged commit 9d9134e into master Feb 21, 2026
3 checks passed
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