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

Matin/FEQ-2526/Add Vite bundle analyzer #227

Merged

matin-deriv merged 1 commit into deriv-com:master from matin-deriv:matin/bundle-analyzer

Jul 31, 2024

Contributor

matin-deriv commented Jul 31, 2024

PR Change:

Added Vite bundle analyzer script
Added title for the iFrames to improve accessibility
Moved Vite-related packages to devDependencies


          chore: Add Vite bundle analyzer

5e2b0e6

matin-deriv requested review from ali-hosseini-deriv, yashim-deriv, azmib-developer and farrah-deriv as code owners

July 31, 2024 04:07

github-actions bot commented Jul 31, 2024

Preview Link: https://matin-bundle-analyze.p2p-standalone.pages.dev

Name	Result
Build status	Completed ✅
Preview URL	Visit Preview
Action URL	Visit Action

farrah-deriv approved these changes

View reviewed changes

coveralls commented Jul 31, 2024

Pull Request Test Coverage Report for Build 10173893647

Details

0 of 0 changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 85.443%

Totals
Change from base Build 10158857559:	0.0%
Covered Lines:	5609
Relevant Lines:	6106

💛 - Coveralls

matin-deriv merged commit dfcc9fc into deriv-com:master

3 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

farrah-deriv farrah-deriv approved these changes

ali-hosseini-deriv Awaiting requested review from ali-hosseini-deriv

yashim-deriv Awaiting requested review from yashim-deriv yashim-deriv is a code owner

azmib-developer Awaiting requested review from azmib-developer

Labels

None yet