rusackas opened a new pull request, #37873:
URL: https://github.com/apache/superset/pull/37873

   ## Summary
   
   Completely removes all Applitools visual testing integration from the 
codebase. We're no longer using this service.
   
   ### Deleted files (10 files)
   - `.github/workflows/superset-applitool-cypress.yml` - Cypress visual 
testing workflow
   - `.github/workflows/superset-applitools-storybook.yml` - Storybook visual 
testing workflow
   - `superset-frontend/applitools.config.js` - Storybook Applitools config
   - `superset-frontend/cypress-base/applitools.config.js` - Cypress Applitools 
config
   - `superset-frontend/cypress-base/cypress/applitools/*.test.ts` - 5 
Applitools test files
   - `docs/static/img/applitools.png` - Logo image
   
   ### Modified files
   - **Dependencies**: Removed `@applitools/eyes-storybook` and 
`@applitools/eyes-cypress`
   - **Cypress config**: Removed eyesPlugin wrapper and applitools spec pattern
   - **Cypress support**: Removed `@applitools/eyes-cypress/commands` import
   - **TypeScript**: Removed Applitools types from tsconfig.json
   - **CI**: Removed `cypress-run-applitools` function from bashlib.sh
   - **License review**: Removed Applitools package exceptions from 
dependency-review.yml
   - **Docs footer**: Removed Applitools logo/link from site footer
   - **Style guide**: Removed Applitools mention from frontend guidelines
   
   ### Impact
   - Removes ~7,800 lines of code and dependencies
   - Removes 2 scheduled CI workflows that were running daily
   - No functionality change - Applitools was only used for visual regression 
testing
   
   ## Test plan
   
   - CI should pass without Applitools
   - Cypress tests should still run (regular e2e tests unaffected)
   - Docs site should build and display correctly without Applitools logo in 
footer
   
   🤖 Generated with [Claude Code](https://claude.ai/code)


-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to