Remove gulp

[knitcodemonkey]

Description

Attempts to replace outdated gulp-tape plugin that prevents contributors from using JSX syntax.

• Replace gulp with package.json scripts.
• Replace jshint with eslint
• Update spacing so we don't have to worry about it in the future
• Remove node versions 6 and 7, which have reached EOL, from yaml file.

This is a breaking change because eslint doesn't support node 6 and below.

Fixes/addresses #178

Type of change

• [X] Breaking change
• [X] New feature (non-breaking change which adds functionality)

Checklist:

• [X] My code follows the style guidelines of this project
• [X] I have performed a self-review of my own code
• [X] I have commented my code, particularly in hard-to-understand areas
• [X] I have made corresponding changes to the documentation
• [X] My changes generate no new warnings
• [ ] I have added tests that prove my fix is effective or that my feature works
• [X] New and existing unit tests pass locally with my changes
• [X] Any dependent changes have been merged and published in downstream modules
• [ ] Changes have been manually tested (please provide information on test platform using the fields below)

Test / Development Platform Information

• Operating system and version
• Avrgirl Arduino version
• NodeJS version
• Arduino Board being used

[coveralls]

Coverage decreased (-0.3%) to 44.211% when pulling ac8873489bbdf9f7ba87ea9ba3f8b25136f2c5ed on knitcodemonkey:remove-gulp into 1dfd9c12f12a3076ad131199952e23543f15e9fd on noopkat:master.

[coveralls]

Coverage decreased (-0.3%) to 44.211% when pulling 7d557d71165460984a03f00403e2eaab1aeb1508 on knitcodemonkey:remove-gulp into 1dfd9c12f12a3076ad131199952e23543f15e9fd on noopkat:master.

[noopkat]

this is really good thank you! I looked at the current NodeJS support timelines and NodeJS versions 6 & 7 are ~not~ now end of life, so I think we can go ahead and:

1. remove them from our travis yaml config
2. bump the next version of avrgirl-arduino to a major release with these changes in it

I'll test with physical hardware before merging this but it should all be fine 🌈

Thanks again for bringing this library into the present day after being stuck in the past.

[knitcodemonkey]

Thank you for looking at this PR, @noopkat. I have removed node 6 and 7 from the yaml file, and bumped the version to 4.0.0. Even if this doesn't bring you all the way up to date, it certainly makes the last of the updates easier!

[noopkat]

hey @knitcodemonkey this doesn't look like it's going to merge properly as far as Github is concerned but thank you for much for removing gulp and leveling up our test and linting framework!