Skip to content
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

Test Upgrade to Serialport 4x #128

Closed
wants to merge 1 commit into from

Conversation

reconbot
Copy link
Member

To be used in testing. By the 4.0.0 release I hope to have browser serialport in parity.

SerialPort 4.0.0-rc1 is out and brings performance improvements, bug fixes and a few changes to the api. It's now out of sync with browser-serialport but not in a major way. garrows/browser-serialport#40

I suspect this upgrade won't have any far reaching unattended effects as firmata uses only a fraction of the functions of SerialPort by default. The coms changes will be cleaner when browser serial port catches up.

To be used in testing. By the 4.0.0 release I hope to have browser serialport in parity.

SerialPort 4.0.0-rc1 is out and brings performance improvements, bug fixes and a few changes to the api. It's now out of sync with `browser-serialport` but not in a major way. garrows/browser-serialport#40

I suspect this upgrade won't have any far reaching effects as firmata uses only a fraction of the functions of SerialPort by default. The coms changes will be cleaner when browser serial port catches up.
@reconbot
Copy link
Member Author

Tests are still not in great shape, they're written to the previous API

@rwaldron
Copy link
Collaborator

rwaldron commented Jul 2, 2016

I resolved this and landed a variant of the upgrade

@rwaldron rwaldron closed this Jul 2, 2016
@reconbot reconbot deleted the serialport-4.0.0-rc1 branch July 2, 2016 17:18
@rwaldron
Copy link
Collaborator

rwaldron commented Jul 2, 2016

dd07058

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.

2 participants