davidzollo commented on PR #10447: URL: https://github.com/apache/seatunnel/pull/10447#issuecomment-5522550165
I pushed one more commit to close out the remaining blocker from the last review round: `5ce9695c5`. The branch had already been synced with `apache/dev` (this PR is now `ahead_by=136`, `behind_by=0`, no merge conflicts), so the only open item left was the documentation-layout blocker Daniel flagged on 2026-07-19 / 2026-08-31: `docs/zh/connectors/source/HubSpot.md` carried a manually-added ASF license header block before the `import ChangeLog from ...` line. `MarkdownTest.testConnectorDocWithChangeLogFlagAndFile` and `testPrimaryHeadersHaveNoTextAbove` (in `seatunnel-ci-tools`) both require line 1 of every connector source/sink doc to be the changelog import — none of the other connector docs (`Github.md`, `Gitlab.md`, `Klaviyo.md`, and this file's own English counterpart) carry that header. I removed the 17-line header block so the file matches the repo-standard layout; the English doc was already correct. Verified locally: `./mvnw spotless:apply -pl seatunnel-connectors-v2/connector-http/connector-http-hubspot -am -nsu` reports no further changes needed, and the diff is exactly the header removal (`git diff` — 17 deletions, no other lines touched). CI is running on the new head now. @Suresh-Krishna-Kusuma no action needed from you unless it turns up something unrelated — I'll keep an eye on the `Build` check. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
