Skip to content
This repository has been archived by the owner on Jul 13, 2019. It is now read-only.

[WIP] chore(app): update react package #47

Open
wants to merge 1 commit into
base: template/react
Choose a base branch
from

Conversation

jeroenptrs
Copy link
Contributor

Description

Updates the react template to improve potential security risks and update performance

Motivation and Context

Removed bloat like husky (precommit works without it) and jss (use a smaller processor like nano-css instead)

How Has This Been Tested?

In the process of testing bundling correctness, size, etc.

Screenshots (if appropriate):

Types of changes

  • Chore (tests, refactors, and fixes)
  • New feature (adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My code follows the code style of this project.
  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • I have read the CONTRIBUTING document.

Closing issues

Fixes #23

@jeroenptrs jeroenptrs added the PR: don't merge This doesn't seem right label Dec 10, 2018
@jeroenptrs jeroenptrs self-assigned this Dec 10, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
PR: don't merge This doesn't seem right
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant