Merge pull request #22 from dwreeves/21-bugfix

[Bugfix] `group_by=` doesn't work for >1 grouped variable

Author: dwreeves

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+### `0.2.6`
+
+- Fix bug with `group_by` on multiple variables; contributed by [@svkohler](https://github.com/dwreeves/dbt_linreg/issues/21).
+
 ### `0.2.5`
 
 - Fix bug where `exog` and `group_by` did not handle `str` inputs e.g. `exog="x"`.

README.md

@@ -32,7 +32,7 @@ Add this the `packages:` list your dbt project's `packages.yml`:
 
 ```yaml
   - package: "dwreeves/dbt_linreg"
-    version: "0.2.5"
+    version: "0.2.6"
 ```
 
 The full file will look something like this:
@@ -43,7 +43,7 @@ packages:
   # Other packages here
   # ...
   - package: "dwreeves/dbt_linreg"
-    version: "0.2.5"
+    version: "0.2.6"
 ```
 
 # Examples

dbt_project.yml

@@ -1,5 +1,5 @@
 name: "dbt_linreg"
-version: "0.2.5"
+version: "0.2.6"
 
 # 1.2 is required because of modules.itertools.
 require-dbt-version: [">=1.2.0", "<2.0.0"]

macros/linear_regression/utils/utils.sql

@@ -221,7 +221,7 @@ inner join {{ join_to }}
 on
   {%- for _ in group_by %}
   {{ join_from }}.gb{{ loop.index }} = {{ join_to }}.gb{{ loop.index }}
-  {%- if not loop.last -%}
+  {% if not loop.last -%}
   and
   {%- endif %}
   {%- endfor %}