Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ci: attach Cypress CT diff-snapshots to CI report #9293

Merged
merged 5 commits into from
Oct 4, 2024
Merged

ci: attach Cypress CT diff-snapshots to CI report #9293

merged 5 commits into from
Oct 4, 2024

Conversation

nsbarsukov
Copy link
Member

@nsbarsukov nsbarsukov commented Oct 1, 2024 •
edited
Loading

Снимок экрана 2024-10-03 в 17 53 18

@nsbarsukov nsbarsukov self-assigned this Oct 1, 2024
@lumberjack-bot lumberjack-bot
Copy link

lumberjack-bot bot commented Oct 1, 2024 •
edited
Loading

Pull request was closed ✔️

All saved screenshots (for current PR) were deleted 🗑️

@bundlemon BundleMon
Copy link

bundlemon bot commented Oct 1, 2024 •
edited
Loading

BundleMon

Unchanged files (5)
Status Path Size Limits
✅ demo/browser/main.(hash).js
 292.67KB +10%
✅ demo/browser/vendor.(hash).js
 247.36KB +10%
✅ demo/browser/runtime.(hash).js
 42.88KB +10%
✅ demo/browser/styles.(hash).css
 16.27KB +10%
✅ demo/browser/polyfills.(hash).js
 11.18KB +10%

Total files change +2B 0%

Unchanged groups (1)
Status Path Size Limits
✅ demo/browser/*..js
 7.1MB -

Final result: ✅

View report in BundleMon website ➡️

Current branch size history | Target branch size history

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Oct 1, 2024 •
edited
Loading

Visit the preview URL for this PR (updated for commit 9aaf191):

https://taiga-previews--pr9293-screenshot-testing-f-w7qm3p14.web.app

(expires Sat, 05 Oct 2024 10:15:38 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 20bbff1ad19772ecf9c5134a84c0641b0d776f4a

@nsbarsukov nsbarsukov force-pushed the screenshot-testing-for-cypress-ct branch 16 times, most recently from 5bd4c84 to 2f686c4 Compare October 2, 2024 13:17
@nsbarsukov nsbarsukov changed the title [WIP] [DONT REVIEW] [DONT MERGE] Just some CI experiments... [WIP] ci: attach Cypress CT diff-snapshots to CI report Oct 2, 2024
@nsbarsukov nsbarsukov force-pushed the screenshot-testing-for-cypress-ct branch 6 times, most recently from 238d6cf to fcbb395 Compare October 3, 2024 07:45
@nsbarsukov nsbarsukov changed the title [WIP] ci: attach Cypress CT diff-snapshots to CI report ci: attach Cypress CT diff-snapshots to CI report Oct 3, 2024
@nsbarsukov nsbarsukov marked this pull request as ready for review October 3, 2024 08:06
@nsbarsukov nsbarsukov changed the title ci: attach Cypress CT diff-snapshots to CI report [WIP] ci: attach Cypress CT diff-snapshots to CI report Oct 3, 2024
@nsbarsukov nsbarsukov force-pushed the screenshot-testing-for-cypress-ct branch 10 times, most recently from 922e847 to 900513d Compare October 3, 2024 14:39
@nsbarsukov nsbarsukov changed the title [WIP] ci: attach Cypress CT diff-snapshots to CI report ci: attach Cypress CT diff-snapshots to CI report Oct 3, 2024
@nsbarsukov nsbarsukov force-pushed the screenshot-testing-for-cypress-ct branch from 900513d to 9aaf191 Compare October 4, 2024 10:09
@nsbarsukov nsbarsukov merged commit 2c9a668 into main Oct 4, 2024
25 of 26 checks passed
@nsbarsukov nsbarsukov deleted the screenshot-testing-for-cypress-ct branch October 4, 2024 10:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vladimirpotekhin vladimirpotekhin vladimirpotekhin approved these changes

@splincode splincode splincode approved these changes

@waterplea waterplea Awaiting requested review from waterplea

@MarsiBarsi MarsiBarsi Awaiting requested review from MarsiBarsi

@mdlufy mdlufy Awaiting requested review from mdlufy

Assignees

@nsbarsukov nsbarsukov

Labels
ready to merge
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@nsbarsukov @splincode @vladimirpotekhin