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

Add scripts #23

Merged
merged 12 commits into from Dec 19, 2018
Merged

Add scripts #23

merged 12 commits into from Dec 19, 2018

Conversation

ImogenF
Copy link
Contributor

@ImogenF ImogenF commented Dec 18, 2018

Removes parcel and makes use of @operational/scripts. PR mainly consists of extensive TS improvements because of stricter TS config in operational-scripts than previously in operational-visualizations.

Main check required is that none of the visual tests or unit tests break.

This makes this repo compatible with explore-ui.

Copy link

@TejasQ TejasQ left a comment

Choose a reason for hiding this comment

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

This is honestly too big to review, but I trust your skills. Let's merge this and deal with any issues in subsequent PRs.

@ImogenF ImogenF merged commit 9bc3bb7 into master Dec 19, 2018
@ImogenF ImogenF deleted the add-scripts branch December 19, 2018 10:13
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

2 participants