Skip to content

Conversation

@utkarsharma2
Copy link
Contributor

The following line when added to the error message and printed to the console -

[yellow]Additional check[/]: If there are deprecated modules in the list, please add them to DEPRECATED_MODULES in scripts/in_container/run_provider_yaml_files_check.py[/]

Results in the below error because of extra [/]:

Traceback (most recent call last):
  File "/Applications/PyCharm CE.app/Contents/plugins/python-ce/helpers/pydev/pydevconsole.py", line 364, in runcode
    coro = func()
  File "<input>", line 1, in <module>
  File "/Users/utkarsharma/sandbox/airflow-sandbox/venv_3.10/lib/python3.10/site-packages/rich/console.py", line 1673, in print
    renderables = self._collect_renderables(
  File "/Users/utkarsharma/sandbox/airflow-sandbox/venv_3.10/lib/python3.10/site-packages/rich/console.py", line 1537, in _collect_renderables
    self.render_str(
  File "/Users/utkarsharma/sandbox/airflow-sandbox/venv_3.10/lib/python3.10/site-packages/rich/console.py", line 1429, in render_str
    rich_text = render_markup(
  File "/Users/utkarsharma/sandbox/airflow-sandbox/venv_3.10/lib/python3.10/site-packages/rich/markup.py", line 168, in render
    raise MarkupError(
rich.errors.MarkupError: closing tag '[/]' at position 562 has nothing to close

Currently console.print failes to print output because of extra  in error message.
@potiuk potiuk merged commit 18eb1fc into apache:main Oct 18, 2023
potiuk pushed a commit that referenced this pull request Oct 29, 2023
Currently console.print failes to print output because of extra  in error message.

(cherry picked from commit 18eb1fc)
@potiuk potiuk added this to the Airflow 2.7.3 milestone Oct 29, 2023
@potiuk potiuk added the changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) label Oct 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dev-tools changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants