davidzollo commented on PR #12300: URL: https://github.com/apache/seatunnel/pull/12300#issuecomment-5677572215
> Well-crafted addition that fixes a genuine gap: CLI Python code previously had no CI coverage at all. The path filtering, matrix setup, `fail-fast: false`, `python -m pytest` choice, and minimal rule selection are all sound. The only substantive concern is the unpinned ruff version, which could cause spurious CI failures on a ruff release without any code change. Hi @utafrali, I‘m so glad to see that you help review the PRs for Apache SeaTunnel Community. Thanks for your contribution! Contributors like you are what keep this community moving forward. Feel free to talk about any issues in daily developing or reviewing — we're always glad to have you here! Please ignore the following message if you have joined in the Slack~ To facilitate focused communication among contributors, we’ve created a brand-new dedicated #contributors channel in official Slack workspace. Slack workspace link: https://s.apache.org/seatunnel-slack This channel is designed for technical discussions, feature design brainstorming, roadmap sync and direct communication with project core Contributors. Everyone who contributes code, docs, test cases or community ideas is warmly invited to join, share thoughts and collaborate together under The Apache Way. Welcome to join 😄 -- 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]
