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

fix(tests): fixup for #1279 #1285

Merged
merged 1 commit into from
Aug 7, 2017
Merged

fix(tests): fixup for #1279 #1285

merged 1 commit into from
Aug 7, 2017

Conversation

reconbot
Copy link
Member

@reconbot reconbot commented Aug 7, 2017

No description provided.

@serialport serialport deleted a comment from codecov-io Aug 7, 2017
@reconbot reconbot merged commit 56074f6 into master Aug 7, 2017
@reconbot reconbot deleted the fix-tests branch August 7, 2017 00:49
reconbot added a commit that referenced this pull request Aug 7, 2017
### Bug Fixes

* **linux:** The productID should be a number not a description string ([#1279](#1279)) ([bf46f68](bf46f68))
* **tests:** fixup for [#1279](#1279) ([#1285](#1285)) ([56074f6](56074f6))
* **windows:** Add option to disable RTS ([#1277](#1277)) ([5b8d163](5b8d163))
* **windows:** Parse more types of pnpIds ([#1288](#1288)) ([0b554d7](0b554d7)), closes [#1220](#1220)


### Chores

* **binaries:** Lets switch to prebuild! ([#1282](#1282)) ([8c36e99](8c36e99))


### Features

* **test:** tone down codecov comments ([#1289](#1289)) ([749ffac](749ffac))


### BREAKING CHANGES

* **binaries:** I'm considering the switch to `prebuild` a breaking change because it's substantially changes our install processes. It's also possible the install flags to ensure downloading or building from source has changed slightly. That's not our api per say, but it's enough.
* **windows:** We previously hard coded to have RTS on for windows at all times it now default to off.
@lock lock bot locked and limited conversation to collaborators Feb 5, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant