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

refactor(looker): build dashboard specs in a utility function #21833

Merged
merged 1 commit into from
May 14, 2024

Conversation

rexledesma
Copy link
Contributor

Summary & Motivation

Addresses #21687 (comment).

How I Tested These Changes

pytest

Copy link
Member

@benpankow benpankow left a comment

Choose a reason for hiding this comment

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

🙏

@rexledesma rexledesma force-pushed the rl/add-looker-api-docs branch from 5776976 to 1af4b5d Compare May 14, 2024 18:24
@rexledesma rexledesma force-pushed the rl/refactor-dashboard-specs-to-utils branch from 00af07a to decd116 Compare May 14, 2024 18:24
@rexledesma rexledesma force-pushed the rl/add-looker-api-docs branch from 1af4b5d to c6d8a68 Compare May 14, 2024 18:41
@rexledesma rexledesma force-pushed the rl/refactor-dashboard-specs-to-utils branch 2 times, most recently from e3e69a4 to 074d5af Compare May 14, 2024 18:49
@rexledesma rexledesma force-pushed the rl/add-looker-api-docs branch from c6d8a68 to 04012ed Compare May 14, 2024 18:58
@graphite-app graphite-app bot requested a review from erinkcochran87 May 14, 2024 18:58
@graphite-app graphite-app bot added the area: docs Related to documentation in general label May 14, 2024
Base automatically changed from rl/add-looker-api-docs to master May 14, 2024 18:59
@rexledesma rexledesma force-pushed the rl/refactor-dashboard-specs-to-utils branch from 074d5af to 66b4808 Compare May 14, 2024 18:59
Copy link
Contributor Author

rexledesma commented May 14, 2024

Merge activity

  • May 14, 11:59 AM PDT: Graphite rebased this pull request after merging its parent, because this pull request is set to merge when ready.
  • May 14, 12:00 PM PDT: @rexledesma merged this pull request with Graphite.

@rexledesma rexledesma merged commit acb0005 into master May 14, 2024
1 check was pending
@rexledesma rexledesma deleted the rl/refactor-dashboard-specs-to-utils branch May 14, 2024 19:00
alangenfeld pushed a commit that referenced this pull request May 14, 2024
## Summary & Motivation
Addresses
#21687 (comment).

## How I Tested These Changes
pytest
danielgafni pushed a commit to danielgafni/dagster that referenced this pull request Jun 18, 2024
…r-io#21833)

## Summary & Motivation
Addresses
dagster-io#21687 (comment).

## How I Tested These Changes
pytest
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: docs Related to documentation in general
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants