Skip to content

docs: exit on yaml parsing error - #239

Merged
justeph merged 1 commit into
mainfrom
docs-fix-torch-and-polars-yaml-files
Aug 19, 2026
Merged

docs: exit on yaml parsing error#239
justeph merged 1 commit into
mainfrom
docs-fix-torch-and-polars-yaml-files

Conversation

@justeph

@justeph justeph commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

The multi-line comment blocks in the polars-runtime-32, polars-runtime-64, and torch YAML files were malformed, causing YAML parsing errors. As a result, these files were silently ignored and were not deployed or included in the documentation.

Fix the malformed comment blocks and exit on parsing errors to prevent similar issues from being silently ignored in the future.

@justeph
justeph requested review from luhenry and threexc August 18, 2026 21:04
@github-actions

github-actions Bot commented Aug 18, 2026

Copy link
Copy Markdown
Contributor
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-08-19 12:13 UTC

@threexc threexc left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Not sure how I missed this.

The multi-line comment blocks in the polars-runtime-32, polars-runtime-64,
and torch YAML files were malformed, causing YAML parsing errors. As a
result, these files were silently ignored and were not deployed or included
in the documentation.

Fix the malformed comment blocks and exit on parsing errors to prevent
similar issues from being silently ignored in the future.

Signed-off-by: Julien Stephan <jstephan@baylibre.com>
@justeph
justeph force-pushed the docs-fix-torch-and-polars-yaml-files branch from f0e92fe to 42e9e52 Compare August 19, 2026 12:01
@justeph
justeph merged commit 6139ccb into main Aug 19, 2026
5 checks passed
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.

3 participants