potiuk opened a new issue, #75: URL: https://github.com/apache/airflow-steward/issues/75
PRs that meaningfully change the UI **and** add Playwright / end-to-end tests should attach a short video demo of the feature working. **Why:** UI test code alone does not communicate the user-visible behavior; reviewers and future archaeology benefit massively from a 20-second clip showing the flow. ### Open questions - **Enforcement:** is this a `CONTRIBUTING.md` policy (human-checked at review time), or a bot-side check that warns / labels PRs touching UI files + adding Playwright specs but lacking a video link? - **"Meaningful UI change":** heuristic for non-trivial vs. trivial CSS tweaks. Touching files in known UI dirs + a non-zero rendered diff? Author-declared via PR template checkbox? - **Hosting:** uploaded to the GitHub PR description, YouTube unlisted, both accepted? - Should the requirement attach to PRs that change UI **without** adding e2e tests too, or only when both are true? -- 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]
