etsy / boundary-layer

Builds Airflow DAGs from configuration files. Powers all DAGs on the Etsy Data Platform
Apache License 2.0
262 stars 58 forks source link

Relax constraints on jinja2 dep #138

Closed li-roy closed 1 year ago

li-roy commented 1 year ago

Description

Relax constraints on jinja2 dep. There don't seem to be any breaking changes from 2.x to 3.0 (https://jinja.palletsprojects.com/en/3.1.x/changes/).

Context / Why are we making this change?

We are running into a dependency conflict. Specifically, flask requires jinja2>=3.0.

Testing and QA Plan

Run tests with recent jinja2.

li-roy commented 1 year ago

@vchiapaikeo Hi, does this project still have active support? If so could you help us with this change? Otherwise please let me know so I can adjust accordingly.

vchiapaikeo commented 1 year ago

Hi @li-roy , we are not actively supporting this project anymore. However, this change seems minor and it probably won't hurt us too much. Let me try to fix the tests and see if we can get this going for you.

vchiapaikeo commented 1 year ago

https://github.com/etsy/boundary-layer/pull/139 should address