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.

Sign up for GitHub

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

Jump to bottom

Fix CLI vars test to check for object instead of string #6850

Merged
merged 2 commits into from
Feb 6, 2023
Merged

Fix CLI vars test to check for object instead of string #6850

merged 2 commits into from
Feb 6, 2023

Conversation

aranke
Copy link
Member

@aranke aranke commented Feb 3, 2023

resolves #

Description

Checklist

@aranke aranke requested a review from a team as a code owner February 3, 2023 00:55
@cla-bot cla-bot bot added the cla:yes label Feb 3, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Feb 3, 2023

Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the contributing guide.

@aranke aranke merged commit 9c0b62b into feature/click-cli Feb 6, 2023
@aranke aranke deleted the fix_cli_vars_test branch February 6, 2023 23:13
This was referenced Feb 8, 2023
Merged
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ChenyuLInx ChenyuLInx ChenyuLInx approved these changes

@MichelleArk MichelleArk MichelleArk approved these changes

@stu-k stu-k Awaiting requested review from stu-k stu-k was automatically assigned from dbt-labs/core-group

@dbeatty10 dbeatty10 Awaiting requested review from dbeatty10 dbeatty10 was automatically assigned from dbt-labs/core-group

Assignees
No one assigned
Labels
cla:yes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@aranke @MichelleArk @ChenyuLInx