DanielLeens commented on PR #10573: URL: https://github.com/apache/seatunnel/pull/10573#issuecomment-6287417204
Status update while going through the open first-time-contributor PRs, @BrandaoFelipe: I resolved the conflicts and the `junit-jupiter-api` test-scope item on your sibling PR #10252, but this PR has "Allow edits by maintainers" disabled, so I cannot push the equivalent fix here. To get this one moving, please: (1) merge the current `dev` and resolve the conflict in `connector-jdbc/pom.xml` (the `yashandb-jdbc` managed dependency was added on `dev` next to your entries; keep both, as separate `<dependency>` blocks); (2) apply the same build fixes as in #10252's `3c2b7afaa5e`: `<scope>test</scope>` on `junit-jupiter-api`, and no nested `junit-bom` import inside `connector-jdbc` (the root POM already imports it, and the nested import breaks `connector-cdc-db2`, which imports `connector-jdbc:pom` from the reactor); (3) enable GitHub Actions on your fork so a Build run attaches. Alternatively, tick "Allow edits by maintainers" on this PR and I can push the same change here. -- 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]
