cloud-custodian / cel-python

Pure Python implementation of the Common Expression Language
Apache License 2.0
99 stars 20 forks source link

Resolve lark-parser dependency version conflicts #25

Closed kylejohnson514 closed 3 years ago

kylejohnson514 commented 3 years ago

Resolve conflicting dependency versions by using lark-parser versions 0.10.1 or newer, rather than a strict requirement of 0.11.2

Downgrading the version of lark-parser doesn't seem to break anything and allows CEL-Python to be implemented in more places, such as Cloud Custodian. Custodian has an underlying dependency on python-hcl2, which requires a 0.10.* version of lark-parser

codecov[bot] commented 3 years ago

Codecov Report

Merging #25 (e685f31) into master (5534d98) will not change coverage. The diff coverage is n/a.

:exclamation: Current head e685f31 differs from pull request most recent head 19b8359. Consider uploading reports for the commit 19b8359 to get more accurate results Impacted file tree graph

@@            Coverage Diff            @@
##            master       #25   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            8         8           
  Lines         2602      2602           
=========================================
  Hits          2602      2602           

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 5534d98...19b8359. Read the comment docs.