Skip to content

Update build and CI config#103

Open
jazeee wants to merge 1 commit intoTom-Alexander:masterfrom
jazeee:update-build
Open

Update build and CI config#103
jazeee wants to merge 1 commit intoTom-Alexander:masterfrom
jazeee:update-build

Conversation

@jazeee
Copy link
Copy Markdown

@jazeee jazeee commented Jan 23, 2020

Fix #102

Comment thread babel.config.js
@@ -0,0 +1,17 @@
const presets = [
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Generally, followed the babel upgrade guidelines.

Comment thread src/regression.js
@@ -1,41 +1,6 @@
const DEFAULT_OPTIONS = { order: 2, precision: 2, period: null };
const { round, deriveDataProperties } = require('./utils');
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

CI said to extract common code into utilities.
It also complained about this file being too large.

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.

Package no longer builds or passes CI

2 participants