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

Merge dev/0.6.5 branch into master ahead of release #369

Merged
merged 15 commits into from
May 18, 2021
Merged

Conversation

clrcrl
Copy link
Contributor

@clrcrl clrcrl commented May 18, 2021

I wanted to move every non-breaking change into a patch release, so that people could upgrade easily without needing to go to 0.20.0. I think this is going to take some of the pressure of the 0.7.0 release.

As part of this cleanup, I also squashed the separate commits from PRs into one single commit, to make this history seem much cleaner.

Once merged and released, I'm going to rebased dev/0.7.0 onto this branch, and clean up the commits that I'm keeping on there. That should put us in great shape to focus on the 0.20.0 changes

Here's the summary of commits that were on the dev/0.7.0 branch, and where I've now represented them :)

original commit hash description now part of: new commit hash reason
2e3a343 Create accepted_range test 0.6.5 3d61f89 Non-breaking feature
6ce6c4d Add integration tests 0.6.5 3d61f89 Non-breaking feature
caa4ca2 Add accepted_range documentation 0.6.5 3d61f89 Non-breaking feature
70b528a Fixing invalid yaml in readme 0.6.5 3d61f89 Non-breaking feature
3ceb9ae Add missing columns: entry 0.6.5 3d61f89 Non-breaking feature
163251d Update changelog for dev/0.7.0 n/a n/a
6e934d7 Added extension to the unique_combination_of_columns schema test in order to support a WHERE arg dropped We get where for free in 0.20.0
0cd7ade Added ability to use expression_is_true as a column test 0.6.5 9d9e720 Non-breaking feature
e5dd0b1 Tidy up Jinja 0.6.5 9d9e720 Non-breaking feature
db44ad3 Update changelog 0.6.5 9d9e720 Non-breaking feature
171970e Allow aliasing individual columns from star macro. dropped General opinion on whether to add complexity to existing macros has changed
edf40bf Fix failing test dropped General opinion on whether to add complexity to existing macros has changed
e590ebb Update changelog [ci skip] dropped General opinion on whether to add complexity to existing macros has changed
bc80c65 Adding some logic to handle unpivoting boolean columns 0.6.5 d4779ca Fix
7c1082d Readme & deleted the 'remove' line which does nothing. 0.6.5 d4779ca Fix
f67875f Finish off upivot boolean logic 0.6.5 d4779ca Fix
3f477bb Update changelog [ci skip] 0.6.5 d4779ca Fix
10bf540 Schema Test - Not accepted values 0.6.5 0101a7e Non-breaking feature
689707d Schema Test - Not accepted values test fixed 0.6.5 0101a7e Non-breaking feature
a3f75be empty lines at the end 0.6.5 0101a7e Non-breaking feature
f60eebc Changelog 0.6.5 0101a7e Non-breaking feature
949d40d Add zero length option to exclusive range test. 0.6.5 63314f6 Non-breaking feature
fb0fa0f Use boolean for zero range arg. 0.6.5 63314f6 Non-breaking feature
5450b71 Fix aliases default arg dropped was related to above change to star macro
4554198 Add fewer_rows_then schema test 0.6.5 e7f02ea Non-breaking feature
b3a7b0e Allow star macro to be case insensitive. Improve docs while we're here. (#348) dropped General opinion on whether to add complexity to existing macros has changed
c304881 Add order_by argument to get_column_values (#349) 0.7.0 TODO Breaking feature (change of arg order)
dd7e24c Postgres also have an alternative concat binary operation (#296) 0.7.0 TODO Potential breaking feature for spark utils
b35f7ed Sequential values v2 (#327) 0.6.5 efc00b Non-breaking feature
8fb77b1 Add slugify macro to utils 0.7.0 TODO Breaking feature (requires dbt 0.19.0 for re module)
603e867 Remove unnecessary parens from pivot macro 0.7.0 TODO Breaking feature (requires dbt 0.19.0 for re module)
14fff40 Handle external tables in get_tables_by_pattern_sql (#351) 0.6.5 6932957 Fix
7c6de17 support quarter in postgres last day (#333) 0.6.5 4f9e717 Fix
56ef1d2 Fix cardinality test error (#334) 0.6.5 8bb8cef Fix
049974c Markdown fixes to fix rendering on hub (#336) 0.6.5 acdd030 Quality of life
f4ef848 Add unit argument to haversine distnace macro 0.6.5 01e9655 Non-breaking feature
09abb04 Update readme and changelog (#361) 0.6.5 5c84f17 Quality of life
3a4da56 Small tidy to docs (#363) 0.6.5 5c84f17 Quality of life

clrcrl and others added 3 commits May 18, 2021 14:57
Add integration tests

Add accepted_range documentation

Fixing invalid yaml in readme

Add missing columns: entry

Co-authored-by: Claire Carroll <carroll.claire.e@gmail.com>

Update changelog
@clrcrl clrcrl force-pushed the dev/0.6.5 branch 3 times, most recently from c022849 to 7a4c5e6 Compare May 18, 2021 20:20
avishalom and others added 5 commits May 18, 2021 16:28
Readme & deleted the 'remove' line which does nothing.

Finish off upivot boolean logic

Update changelog [ci skip]
Schema Test - Not accepted values, test fixed

empty lines at the end

Update changelog
Use boolean for zero range arg.

Update changelog
@clrcrl clrcrl requested a review from jtcohen6 May 18, 2021 21:44
@clrcrl clrcrl changed the title Dev/0.6.5 Merge dev/0.6.5 branch into master ahead of release May 18, 2021
@clrcrl clrcrl marked this pull request as ready for review May 18, 2021 21:47
Copy link
Contributor

@jtcohen6 jtcohen6 left a comment

Choose a reason for hiding this comment

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

Amazing description + commit history! I know these changes have been combed through a few times now.

@clrcrl clrcrl merged commit 909f559 into master May 18, 2021
@joellabes joellabes deleted the dev/0.6.5 branch October 21, 2021 22:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants