Skip to content

chore: clean up attribution and test data whitespace#447

Merged
lxy-9602 merged 2 commits into
alibaba:mainfrom
zjw1111:codex/trim-test-data-readme-whitespace
Jul 27, 2026
Merged

chore: clean up attribution and test data whitespace#447
lxy-9602 merged 2 commits into
alibaba:mainfrom
zjw1111:codex/trim-test-data-readme-whitespace

Conversation

@zjw1111

@zjw1111 zjw1111 commented Jul 24, 2026

Copy link
Copy Markdown
Collaborator

Purpose

Linked issue: N/A

This change:

  • ports the license and notice attribution cleanup from
    apache/paimon-cpp@2e94a8f2f4e2700051f3808c60f39d92fb8e6832;
  • preserves the Alibaba project identity, the existing Alibaba copyright
    headers in the RapidJSON utility and its test, and the current build
    configuration while adding the applicable third-party attribution details
    and license texts; and
  • removes trailing whitespace from README files under test/test_data.

Tests

  • pre-commit run --files <changed-files>
  • git -c core.whitespace=cr-at-eol diff upstream/main...HEAD --check

No build or runtime tests were run because the changes are limited to
attribution metadata, source comments, and trailing whitespace.

API and Format

No public API, storage format, or protocol changes.

Documentation

No user-facing feature or documentation behavior changes.

Generative AI tooling

Generated-by: OpenAI Codex (GPT-5)

@zjw1111
zjw1111 force-pushed the codex/trim-test-data-readme-whitespace branch from c349267 to d0d25e8 Compare July 24, 2026 05:05
@zjw1111
zjw1111 force-pushed the codex/trim-test-data-readme-whitespace branch from d0d25e8 to a4673ba Compare July 24, 2026 11:04

@lxy-9602 lxy-9602 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.

+1

@lxy-9602
lxy-9602 merged commit 2fff022 into alibaba:main Jul 27, 2026
14 of 16 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.

2 participants