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

Use AttrDict instead of agate #604

Merged
merged 11 commits into from
Jan 26, 2023
Merged

Use AttrDict instead of agate #604

merged 11 commits into from
Jan 26, 2023

Conversation

colin-rogers-dbt
Copy link
Contributor

@colin-rogers-dbt colin-rogers-dbt commented Jan 25, 2023 •
edited
Loading

Updating in response to core change to swap from using agate to AttrDict. Core change: dbt-labs/dbt-core#6645

Description

Checklist

@cla-bot cla-bot bot added the cla:yes label Jan 25, 2023
@github-actions
Copy link
Contributor

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 dbt-spark contributing guide.

@colin-rogers-dbt colin-rogers-dbt self-assigned this Jan 25, 2023
@colin-rogers-dbt colin-rogers-dbt added the Skip Changelog Skips GHA to check for changelog file label Jan 25, 2023
@colin-rogers-dbt colin-rogers-dbt changed the title Use AttrDict Use AttrDict instead of agate Jan 26, 2023
@colin-rogers-dbt colin-rogers-dbt merged commit 3114331 into main Jan 26, 2023
@colin-rogers-dbt colin-rogers-dbt deleted the addMyPyExceptionForList branch January 26, 2023 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@McKnight-42 McKnight-42 McKnight-42 approved these changes

@nssalian nssalian Awaiting requested review from nssalian

Assignees

@colin-rogers-dbt colin-rogers-dbt

Labels
cla:yes Skip Changelog Skips GHA to check for changelog file
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@colin-rogers-dbt @McKnight-42