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

release candidate: v1.7 #43

Merged
merged 2 commits into from
Nov 17, 2023
Merged

release candidate: v1.7 #43

merged 2 commits into from
Nov 17, 2023

Conversation

devmozao
Copy link
Collaborator

@devmozao devmozao commented Nov 17, 2023

All the changes at this moment:

  • upgrade vite to v5
  • improved bundle optimization splitting chunks that are common and doesn't change much, like react, react-dom and react/jsx
  • add treemap bundle analyzer
  • improved linters
  • enforced linters before build: it must succeed before commit and deploying with actions
  • removed dep @fvilers/disable-react-devtools and implemented the snippet directly on index.html at root, in order to optimize bundle size
  • improved some script commands to improve DX
  • minor change on .eslintrc in order to prepare to the v9 changes (it will be huge)
  • upgrade to node v20 (🥳🎉)

to-do-in-sometime-in-future:

  • improve eslint rules by visiting each rule and each plugin and enforce them in a friendly way
  • implement strict typescript lint and check with tsc itself
  • apply optimizations for assets compression (brotli, gzip)
  • apply optimizations for other assets like images and fonts (specially fonts injected by third parties)
  • implement qr-code to open application on mobile and facilitate dev testing for responsives
  • check msw workflow with vitest/testing-lib
  • check if css modules will work in chunks/modules as desired

@devmozao devmozao requested a review from raisiqueira November 17, 2023 00:50
@devmozao devmozao self-assigned this Nov 17, 2023
@devmozao devmozao changed the title chore(rc/1.7): release candidate: v1.7 Nov 17, 2023
Copy link
Member

@raisiqueira raisiqueira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🎉

@devmozao
Copy link
Collaborator Author

hype

@devmozao devmozao merged commit 44e247b into main Nov 17, 2023
1 check passed
@raisiqueira raisiqueira deleted the rc/1.7 branch November 17, 2023 23:58
@devmozao devmozao restored the rc/1.7 branch December 11, 2023 22:27
@devmozao devmozao deleted the rc/1.7 branch January 26, 2024 18:51
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.

2 participants