-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
(#5070) - stop testing node 0.10 in CI #5070
Conversation
ping @gr2m what Node versions is Hoodie targeting right now? |
Hoodie is built on hapi which requires node v4+ |
Cool, I'm gonna merge this then. |
Nooo!!! We are completely dependent on Node 10 for Thali. Without Node 10 we cannot ever upgrade PouchDB!!!!! |
BTW, I don't have good data on how many people still use node 0.10 but as just a hint, https://semaphoreci.com/blog/2015/12/15/nodejs-version-usage-in-commercial-projects-2015-edition.html, it's not a small number. |
We're not going to explicitly kill 0.10 support; we're just not going to test it anymore. So it's a YMMV situation. |
Nothing rots faster than code. If we aren't testing on 0.10 then we don't work on 0.10. We can't ship with YMMV. :( |
BTW, I realize this is our problem, not Pouch's but there is still a lot of 0.10 out there. :( |
Bummer. We're working on node 0.10 (via jxcore) and were waiting for #5018 to land (a fix we need). Wish we could have gotten one more PouchDB release with this fix before testing was dropped on Node 0.10. |
@mohlsen Again, it's not that PouchDB no longer works in Node 0.10. We're doing nothing to intentionally break it. I would be happy to continue supporting 0.10, but currently "supporting" means a lot of manual overhead for committers, i.e. mostly me sitting there tapping "refresh" on Travis until the build restarts. If someone can figure out where these intermittent errors are coming from and fix them, then I would be more than happy to add 0.10 testing back in! :) |
So we have a bit of an undocumented process for this, if something is in CI and fails our build it needs an owner, I would be happy to 👍 putting .10 back into allowed failures and if it picks up an owner and becomes stable then happy to remove it from allowed failures |
I'd feel more comfortable if it were in the allow failures so it's easier for someone to see and fix. |
I would have put it in allowed failures, but TBH couldn't figure out how. :/ I'd be happy if someone made that change, though. |
Yeah was going to put a PR in but I'm not sure too, @daleharvey any ideas? :) |
we might have some resources to look into this later this week. |
btw we added |
We appreciate this. We'll need to figure out how to monitor and fix as appropriate. |
We should probably start a discussion about whether to keep
Node 0.10 support or not, but I'd be in favor of just
removing it because it's constantly failing in Travis, and it's
annoying to have to deal with.
We could add it to allowed failures, but I figure why eat
up Travis CPU when we don't need to.