Skip to content
This repository has been archived by the owner on Oct 11, 2020. It is now read-only.

fix: Update dependencies, snapshots, folder structure #17

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

fix: Update dependencies, snapshots, folder structure #17

wants to merge 9 commits into from

Conversation

tpscrpt
Copy link

@tpscrpt tpscrpt commented Jan 1, 2019

  • All speced functionally is implemented and is working
  • Unit tests for all functions and modules
  • Integration tests for all use cases
  • Comment documentation for all modules, functions and types
  • Ready for code review

There were errors relating to ReactDOM which are fixed in a newer version of react-native-web. I've update the react dependencies and changed the snapshots accordingly.

I've also changed the README.md images to their own folder, to remove a bit of clutter from the repo root.

Lastly, the coverage folder is now not included into version control. I recommend setting up a CI pipeline for coverage and integration testing.

@tpscrpt tpscrpt mentioned this pull request Jan 1, 2019
5 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant