Closed aerorahul closed 6 months ago
Attention: Patch coverage is 84.21053%
with 6 lines
in your changes are missing coverage. Please review.
Project coverage is 48.87%. Comparing base (
d69dbbc
) to head (968cddd
).
Files | Patch % | Lines |
---|---|---|
src/wxflow/jinja.py | 86.20% | 3 Missing and 1 partial :warning: |
src/wxflow/schema.py | 0.00% | 1 Missing :warning: |
src/wxflow/yaml_file.py | 87.50% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Description The current implementation of jinja2 rendering was unable to support
include
when the included template was in a location not directly under the current template directory. This PR:searchpath
as well as from/
$( )
from functionparse_j2yaml
. This was a stop gap measure to allow prototyping in the DA applications. Those have now matured that this stop gap needed to be addressed holistically.jinja2.py
andyaml_file.py
. Test for theinclude
is added.Type of change
How Has This Been Tested?
include
including nesting.Checklist