-
-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding CommonJS target on lib folder #86
Conversation
* Exctract barChart fixtures from testData * Extract donutChart fixtures from testData * Extract legendChart fixtures from testData, also rename to legendChart * Extract stackedAreaChart fixtures from testData * Fix LineComponent data proptype * Update requires to the new fixtures in documentation files
yarn.lock
Outdated
@@ -1365,9 +1365,27 @@ braces@^1.8.2: | |||
preserve "^0.2.0" | |||
repeat-element "^1.1.2" | |||
|
|||
<<<<<<< HEAD |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Merge conflicts?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, good call. Fixing
838436d
to
26ffa66
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good for now.
We should consider looking at how recharts does it since it's much simpler and I think they get the same result we want.
https://github.com/recharts/recharts/blob/master/package.json
https://github.com/recharts/recharts/blob/master/webpack.config.js
Not sure yet if it will apply 100% to ours but it could be worth exploring.
* try cleaning up travis yml * Bumped Project to 0.3.17 * try again * Bumped Project to 0.3.18 * go back to manual push * Bumped Project to 0.3.19 * bump britecharts * Bumped Project to 0.3.20 * bump to britecharst 2.4.6 * Bumped Project to 0.3.21 * try ghpages deploy with folder setting * Bumped Project to 0.3.22 * fix syntax * Bumped Project to 0.3.23 * Add Line chart (#72) * Extract fixtures from testData to its own folder (#76) * Exctract barChart fixtures from testData * Extract donutChart fixtures from testData * Extract legendChart fixtures from testData, also rename to legendChart * Extract stackedAreaChart fixtures from testData * Fix LineComponent data proptype * Update requires to the new fixtures in documentation files * Upgrade to latest britecharts * Adding CommonJS modules and updating esm configuration to use commonjs-modules * Fixing merge conflicts in lock file
Description
Adding a commonjs version of the individual files into our lib folder
Updating the format of the esm modules
Motivation and Context
Fixes https://github.com/eventbrite/britecharts-react/issues/67
Tries to get close to solve https://github.com/eventbrite/britecharts/issues/429
How Has This Been Tested?
Ran tests, checked outputs manually
Types of changes