Skip to content

Conversation

@CuteChuanChuan
Copy link
Contributor

Which issue does this PR close?

Part of #16915

Rationale for this change

Format code examples in documentation comments to improve readability and maintain consistent code style across the codebase. This is part of a multi-PR effort to format all doc comment examples and eventually enable CI checks to enforce this formatting.

What changes are included in this PR?

Run cargo fmt -p <crate> -- --config format_code_in_doc_comments=true for the following datasource-related crates:

  • datafusion-datasource
  • datafusion-datasource-arrow
  • datafusion-datasource-avro
  • datafusion-datasource-csv
  • datafusion-datasource-json
  • datafusion-datasource-parquet

Are these changes tested?

No testing needed - this is purely a formatting change with no functional modifications.

Are there any user-facing changes?

No - this only affects documentation formatting.

@github-actions github-actions bot added the datasource Changes to the datasource crate label Oct 28, 2025
Copy link
Contributor

@alamb alamb left a comment

Choose a reason for hiding this comment

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

Thanks @CuteChuanChuan -- can you please also double check the ascii art isn't messed up by this change?

Copy link
Contributor

@alamb alamb left a comment

Choose a reason for hiding this comment

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

Thank you @CuteChuanChuan

@alamb alamb added this pull request to the merge queue Oct 29, 2025
Merged via the queue into apache:main with commit 787c5bc Oct 29, 2025
28 checks passed
@CuteChuanChuan CuteChuanChuan deleted the raymond/16915-format-datasource branch October 29, 2025 23:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

datasource Changes to the datasource crate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants