Re: [PR] Add LiteralValue wrapper rendered without templating [airflow]

2023-11-17 Thread via GitHub
uranusjr merged PR #35017: URL: https://github.com/apache/airflow/pull/35017 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@airflow

Re: [PR] Add LiteralValue wrapper rendered without templating [airflow]

2023-11-16 Thread via GitHub
uranusjr commented on code in PR #35017: URL: https://github.com/apache/airflow/pull/35017#discussion_r1396735049 ## docs/apache-airflow/core-concepts/operators.rst: ## @@ -158,37 +158,50 @@ See the `Jinja documentation

Re: [PR] Add LiteralValue wrapper rendered without templating [airflow]

2023-11-15 Thread via GitHub
uranusjr commented on code in PR #35017: URL: https://github.com/apache/airflow/pull/35017#discussion_r1395224033 ## docs/apache-airflow/core-concepts/operators.rst: ## @@ -158,37 +158,48 @@ See the `Jinja documentation

Re: [PR] Add LiteralValue wrapper rendered without templating [airflow]

2023-11-12 Thread via GitHub
uranusjr commented on code in PR #35017: URL: https://github.com/apache/airflow/pull/35017#discussion_r1390706598 ## docs/apache-airflow/core-concepts/operators.rst: ## @@ -185,10 +185,23 @@ This will fail with ``TemplateNotFound: cat script.sh``, but we can prevent airf

Re: [PR] Add LiteralValue wrapper rendered without templating [airflow]

2023-11-12 Thread via GitHub
michalsosn commented on code in PR #35017: URL: https://github.com/apache/airflow/pull/35017#discussion_r1390545309 ## airflow/template/templater.py: ## @@ -155,6 +167,8 @@ def render_template( return self._render(template, context) if isinstance(value, Res

Re: [PR] Add LiteralValue wrapper rendered without templating [airflow]

2023-11-05 Thread via GitHub
uranusjr commented on code in PR #35017: URL: https://github.com/apache/airflow/pull/35017#discussion_r1382744257 ## airflow/template/templater.py: ## @@ -155,6 +167,8 @@ def render_template( return self._render(template, context) if isinstance(value, Resol

Re: [PR] Add LiteralValue wrapper rendered without templating [airflow]

2023-10-24 Thread via GitHub
michalsosn commented on code in PR #35017: URL: https://github.com/apache/airflow/pull/35017#discussion_r1370785630 ## tests/template/test_templater.py: ## @@ -60,3 +60,23 @@ def test_render_template(self): templater.template_ext = [".txt"] rendered_content = t

Re: [PR] Add LiteralValue wrapper rendered without templating [airflow]

2023-10-23 Thread via GitHub
uranusjr commented on code in PR #35017: URL: https://github.com/apache/airflow/pull/35017#discussion_r1368238095 ## tests/template/test_templater.py: ## @@ -60,3 +60,23 @@ def test_render_template(self): templater.template_ext = [".txt"] rendered_content = tem

Re: [PR] Add LiteralValue wrapper rendered without templating [airflow]

2023-10-23 Thread via GitHub
uranusjr commented on code in PR #35017: URL: https://github.com/apache/airflow/pull/35017#discussion_r1368237626 ## tests/template/test_templater.py: ## @@ -60,3 +60,23 @@ def test_render_template(self): templater.template_ext = [".txt"] rendered_content = tem

Re: [PR] Add LiteralValue wrapper rendered without templating [airflow]

2023-10-23 Thread via GitHub
uranusjr commented on code in PR #35017: URL: https://github.com/apache/airflow/pull/35017#discussion_r1368236102 ## airflow/utils/template.py: ## @@ -0,0 +1,32 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements. See the N

Re: [PR] Add LiteralValue wrapper rendered without templating [airflow]

2023-10-18 Thread via GitHub
uranusjr commented on code in PR #35017: URL: https://github.com/apache/airflow/pull/35017#discussion_r1364213778 ## docs/apache-airflow/core-concepts/operators.rst: ## @@ -185,10 +185,22 @@ This will fail with ``TemplateNotFound: cat script.sh``, but we can prevent airf

Re: [PR] Add LiteralValue wrapper rendered without templating [airflow]

2023-10-18 Thread via GitHub
michalsosn commented on code in PR #35017: URL: https://github.com/apache/airflow/pull/35017#discussion_r1363840911 ## airflow/template/templater.py: ## @@ -167,6 +179,8 @@ def render_template( return {k: self.render_template(v, context, jinja_env, oids) for k, v i

Re: [PR] Add LiteralValue wrapper rendered without templating [airflow]

2023-10-18 Thread via GitHub
Taragolis commented on code in PR #35017: URL: https://github.com/apache/airflow/pull/35017#discussion_r1363820365 ## airflow/template/templater.py: ## @@ -167,6 +179,8 @@ def render_template( return {k: self.render_template(v, context, jinja_env, oids) for k, v in

[PR] Add LiteralValue wrapper rendered without templating [airflow]

2023-10-18 Thread via GitHub
michalsosn opened a new pull request, #35017: URL: https://github.com/apache/airflow/pull/35017 Closes: #33694 The PR introduces the airflow.template.templater.LiteralValue class, which wraps a single value and makes the Templater immediately return, bypassing Jinja templ