Skip to content

Conversation

@sebix
Copy link
Member

@sebix sebix commented Mar 5, 2022

jinja2 version 3 did not introduce breaking changes, so we can remove
the version restriction
add the requirement also for the jinja expert
fix error message on the required library in the jinja expert

jinja2 version 3 did not introduce breaking changes, so we can remove
the version restriction
add the requirement also for the jinja expert
fix error message on the required library in the jinja expert
@codecov-commenter
Copy link

codecov-commenter commented Mar 5, 2022

Codecov Report

Merging #2158 (05c8ca6) into develop (af60b58) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff            @@
##           develop    #2158   +/-   ##
========================================
  Coverage    76.34%   76.34%           
========================================
  Files          441      441           
  Lines        23649    23649           
  Branches      3739     3739           
========================================
  Hits         18055    18055           
  Misses        4857     4857           
  Partials       737      737           
Impacted Files Coverage Δ
intelmq/bots/experts/jinja/expert.py 75.75% <0.00%> (ø)

@sebix
Copy link
Member Author

sebix commented Mar 5, 2022

Will be merged as part of #2155 as we can't merge the other PR otherwise

@sebix sebix closed this Mar 5, 2022
@sebix sebix deleted the fix-jinja2 branch July 17, 2023 07:34
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.

2 participants