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

Move time spine and fill nulls rendering tests to separate module #840

Merged
merged 1 commit into from
Nov 3, 2023

Conversation

tlento
Copy link
Contributor

@tlento tlento commented Nov 3, 2023

With this, the splitting up of test_dataflow_to_sql_plan into
direct tests of the object conversion rendering (still in
test_dataflow_to_sql_plan) and more feature-specific query
rendering (now in the tests/query_rendering package) is complete.

With this, the splitting up of test_dataflow_to_sql_plan into
direct tests of the object conversion rendering (still in
test_dataflow_to_sql_plan) and more feature-specific query
rendering (now in the tests/query_rendering package) is complete.
1 similar comment
Copy link

github-actions bot commented Nov 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.

@tlento tlento changed the title Move time spine and fill nulls rendering tests Move time spine and fill nulls rendering tests to separate module Nov 3, 2023
@tlento tlento added the Run Tests With Other SQL Engines Runs the test suite against the SQL engines in our target environment label Nov 3, 2023
@tlento tlento temporarily deployed to DW_INTEGRATION_TESTS November 3, 2023 00:33 — with GitHub Actions Inactive
@tlento tlento temporarily deployed to DW_INTEGRATION_TESTS November 3, 2023 00:33 — with GitHub Actions Inactive
@tlento tlento temporarily deployed to DW_INTEGRATION_TESTS November 3, 2023 00:33 — with GitHub Actions Inactive
@tlento tlento temporarily deployed to DW_INTEGRATION_TESTS November 3, 2023 00:33 — with GitHub Actions Inactive
@tlento tlento marked this pull request as ready for review November 3, 2023 00:35
@github-actions github-actions bot removed the Run Tests With Other SQL Engines Runs the test suite against the SQL engines in our target environment label Nov 3, 2023
@tlento tlento requested a review from WilliamDee November 3, 2023 00:38
Base automatically changed from move-date-part-rendering-tests to main November 3, 2023 02:19
@tlento tlento merged commit b0b4ccf into main Nov 3, 2023
46 of 48 checks passed
@tlento tlento deleted the 11-02-Move_time_spine_and_fill_nulls_rendering_tests branch November 3, 2023 02:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants