-
Notifications
You must be signed in to change notification settings - Fork 150
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
Check if citgm is clean on Node master #660
Comments
|
|
I added sqlite3, leveldown and level to the list in nodejs/node#25060 |
New run with version 5.0.2: https://ci.nodejs.org/view/Node.js-citgm/job/citgm-smoker/1848/ |
New run with citgm@5.0.8: https://ci.nodejs.org/view/Node.js-citgm/job/citgm-smoker/2075/ |
|
I believe the csv-parser failures will go away once a new version of csv-parser is published. The fix for it is in master, but not in a release. /ping @shellscape |
My OSS backlog is massive right now. A house renovation and 8 month old teething transition to solid food crazy poop child is taking up most of my free time. I'm picking things off as I have the chance to. Gonna be a bit folks. |
serialport is expected to work with master when 8.x comes out of beta. Not sure what the expected release date is. /ping @reconbot |
For zeromq, from nodejs/node#29504 (comment):
|
FWIW here's another run against master (used to test some changes to the CI job): https://ci.nodejs.org/job/citgm-smoker/2078/ |
@Trott got the release out today |
Here's another one: https://ci.nodejs.org/job/citgm-smoker/2079/
|
csv-parser has been updated |
https://ci.nodejs.org/view/Node.js-citgm/job/citgm-smoker/2085/
|
https://ci.nodejs.org/view/Node.js-citgm/job/citgm-smoker/2086/
|
Closing as stale and superseded by the most recent review in #997 |
CI run: https://ci.nodejs.org/view/Node.js-citgm/job/citgm-smoker/1688/
I expect it to fail on Windows because of #652 but if other platforms are red, we should look into the failures.
The text was updated successfully, but these errors were encountered: