Skip to content

Conversation

@oscarlorentzon
Copy link
Member

@oscarlorentzon oscarlorentzon commented Mar 15, 2021

Motivation

  • Treeshake to decrease library size
  • Distribute ES module build, source maps for all builds, type declarations
  • Run test suite independently from browser environments

Contribution

  • Migrate to Rollup
    • Emit umd, minified umd, module, type declarations and source maps
    • Work around problems with Falcor by using virtual module
  • Migrate to Jest
    • Run tests in with CommonJS modules because ES6 is not supported
    • Work around dependency problems by bootstrap to ensure they are not loaded during test
  • Remove all unused dependencies
  • Create state transition matrix to avoidcircular dependencies

Test Plan

yarn build
yarn test
yarn start

Migrate to Rollup.
Emit umd, minified umd, module, type declarations
and source maps.
Work around problems with Falcor by using
virtual module.

Migrate to Jest.
Run tests in with CommonJS modules becasue ES6
is not supported.
Work around dependency problems by bootstrap
to ensure they are not loaded during test.

Remove all unused dependencies.

Create state transition matrix to avoid
circular dependencies.

chore: tsc and rollup watch build
chore: incremental watch build
Remove unused dependencies.
chore: use existing type packages
chore: mock favicon
chore: improve watch build
chore: fix styles build
chore: specific commonjs build for tests
chore: reset spec files
refactor: explicit state transition matrix
chore: make src commands work
chore: working unit tests and lib
chore: add serve docs script
chore: remove unused bundle config
chore: refactor rollup config to avoid duplication
fix: falcor imports
fix: transition from source to target

Fixes #87
Fixes #100
Fixes #159
Fixes #208
@oscarlorentzon oscarlorentzon added this to the 4.0.0 milestone Mar 15, 2021
@oscarlorentzon oscarlorentzon merged commit b562a0b into main Mar 15, 2021
@oscarlorentzon oscarlorentzon deleted the feature-rollup-squash branch March 15, 2021 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants