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

updated conversion tests to include child types #93

Merged
merged 3 commits into from
Dec 21, 2021

Conversation

McKnight-42
Copy link
Contributor

@McKnight-42 McKnight-42 commented Dec 21, 2021

resolves #

Description

co-author: @kwigley

updated test to check for child types, was causing python 3.7 and 3.8 failures

Checklist

  • I have signed the CLA
  • I have run this code in development and it appears to resolve the stated issue
  • This PR includes tests, or tests are not required/relevant for this PR
  • I have updated the CHANGELOG.md and added information about my change to the "dbt-bigquery next" section.

@McKnight-42 McKnight-42 self-assigned this Dec 21, 2021
CHANGELOG.md Outdated Show resolved Hide resolved
added reference to issue

Co-authored-by: Kyle Wigley <kyle@fishtownanalytics.com>
@McKnight-42 McKnight-42 merged commit 05a4364 into main Dec 21, 2021
@McKnight-42 McKnight-42 deleted the McKnight/conversion_test_update branch December 21, 2021 22:16
@@ -10,6 +10,7 @@

### Under the hood
- Adding `execution_project` to `target` object ([#66](https://github.com/dbt-labs/dbt-bigquery/issues/66))
- Fix test related to preventing coercion of boolean values (True, False) to numeric values (0, 1) in query results ([#93](https://github.com/dbt-labs/dbt-bigquery/issues/93))
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we need to change the section this is in @McKnight-42. Could you open another PR that creates a new section above dbt-bigquery 1.1.0 (Release TBD) and then change the release date for dbt-bigquery 1.0.0 to be December 3, 2021? I think we missed updating this when we did the release

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@leahwicz Do we still want it in the under the hood section of the tbd or placed in fixes?

Copy link
Contributor

Choose a reason for hiding this comment

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

Let's start a new Fixes section under 1.1.0 and put this under there

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done, changes made in a new pr #95

McKnight-42 added a commit that referenced this pull request Jan 3, 2022
#95)

* updating changlog, adding tbd for 1.1.0, updating 1.0.0 release date moving #93

* updating changelog to be Fixes section instead of Under the hood
@jtcohen6
Copy link
Contributor

jtcohen6 commented Jan 4, 2022

@McKnight-42 Big belated thanks for this!

There's an open PR in Snowflake that fixes the same test. Think you might be able to take a look? dbt-labs/dbt-snowflake#76

I also think we'll need to make the change over in dbt-redshift, which has the same tests failing.

siephen pushed a commit to AgencyPMG/dbt-bigquery that referenced this pull request May 16, 2022
* updated conversion tests to include child types

* added changelog.md

* Update CHANGELOG.md

added reference to issue

Co-authored-by: Kyle Wigley <kyle@fishtownanalytics.com>

Co-authored-by: Kyle Wigley <kyle@fishtownanalytics.com>
siephen pushed a commit to AgencyPMG/dbt-bigquery that referenced this pull request May 16, 2022
dbt-labs#95)

* updating changlog, adding tbd for 1.1.0, updating 1.0.0 release date moving dbt-labs#93

* updating changelog to be Fixes section instead of Under the hood
leahwicz pushed a commit that referenced this pull request Jun 8, 2022
* updated conversion tests to include child types

* added changelog.md

* Update CHANGELOG.md

added reference to issue

Co-authored-by: Kyle Wigley <kyle@fishtownanalytics.com>

Co-authored-by: Kyle Wigley <kyle@fishtownanalytics.com>
@leahwicz leahwicz mentioned this pull request Jun 8, 2022
4 tasks
leahwicz added a commit that referenced this pull request Jun 15, 2022
* Pointing the 1.0.latest branch to the Core release branch

* updated conversion tests to include child types (#93)

* updated conversion tests to include child types

* added changelog.md

* Update CHANGELOG.md

added reference to issue

Co-authored-by: Kyle Wigley <kyle@fishtownanalytics.com>

Co-authored-by: Kyle Wigley <kyle@fishtownanalytics.com>

* Fixing pip update for CI (#196)

* Fixing whitespace

* Adding missing py39 version from test matrix

Co-authored-by: Matthew McKnight <91097623+McKnight-42@users.noreply.github.com>
Co-authored-by: Kyle Wigley <kyle@fishtownanalytics.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants