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.

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

performance improvements - webpack, entrypoint #26

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

christin-wednesday
Copy link
Contributor

Description


  • Added support for CSS extraction and minification
  • SVG loading
  • Reduced entry point size by moving out Providers
  • Added NPM config variables in scripts

Checklist

  • PR description included
  • yarn test passes
  • Tests are [changed or added]
  • Relevant documentation is changed or added (and PR referenced)

GIF's


refactor: add Provider component to reduce entry point size
refactor: reordered scripts
refactor: add npm-run-all for sequential scripts

Signed-off-by: Chris <[email protected]>
@github-actions
Copy link

github-actions bot commented May 1, 2022

Jest Coverage Report

Total coverage

Status Category Percentage Covered / Total
🟢 Statements 98.21% (-1.79% 🔻) 385/392
🟢 Branches 100% 143/143
🟢 Functions 97.95% (-2.05% 🔻) 143/146
🟢 Lines 97.92% (-2.08% 🔻) 330/337

Status of coverage: 🟢 - ok, 🟡 - slightly more than threshold, 🔴 - under the threshold

Show new covered files 🌑

Coverage of new files

Status Filename Statements Branches Functions Lines
🔴 containers/LanguageProvider/loadable.ts 0% 100% 0% 0%
🔴 containers/App/loadable.ts 0% 100% 0% 0%
🔴 components/Providers/index.tsx 0% 100% 0% 0%

Status of coverage: 🟢 - ok, 🟡 - slightly more than threshold, 🔴 - under the threshold

Report generated by 🧪jest coverage report action from 064b60f

fix: preload font
fix: analyze script arguments

Signed-off-by: Chris <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant