-
Notifications
You must be signed in to change notification settings - Fork 13
Bar #55
base: master
Are you sure you want to change the base?
Bar #55
Conversation
- License and readme created - First code from the draft we had - Linters are disabled for now, that will need to be fixed later
First draft of the vanilla bar chart
Hey @abbystarnes, there are a bunch of |
@seejamescode Thanks for catching that. Not sure how I added the line files, but I've removed them. I rewrote the bar chart based on the line chart so that they'd be in better alignment. What can I do to pass Travis CI tests? |
@abbystarnes Travis looks like it is failing on building the line chart. There is still a |
@seejamescode I've removed the line folder. Travis CI still seems to be getting stuck on variables that are declared in one file and used in another. (no-unused-vars and no-undef). Is there a work around for this? |
I'm taking a look at the build errors. |
* Remove unneeded dist folders * SCSS linter fixes * Fix the compiling errors - Remove extraneous file charts-bar.js and its reference in package.json - Fix linter errors in ivm-charts-bar.js
Changed