Comments (11)
@consideRatio you're right that having tests makes it easier to release.
I opened #93 with an integration test that tests everything except novnc. It needs debugging for why it is failing in github actions, but works great on my (much faster) local machine. I hope that helps.
from jupyter-remote-desktop-proxy.
I'd like to see this major release ship with the breaking change of #91 as well, but with regards to testing I'm OK with the current state having a basic tiger + turbo test now.
Reamining:
from jupyter-remote-desktop-proxy.
fwiw, I think #68 is going to be pretty fairly involved, and I don't think my near future schedule has time for that. I'm happy to provide code review if someone else wants to do that.
I also don't think #81 should block the release.
from jupyter-remote-desktop-proxy.
My personal perspective is that we should really have very low barriers to release, as that helps get the software out to more people's hands - and that's the primary thing that potentially brings in new contributors, which is the lifeblood of keeping a project going and unburntout. Release early and release very often.
from jupyter-remote-desktop-proxy.
Release early and often is something I want too, and I find tests make it easier to do so. Contributing to a project would smoother for PR authors and maintainers if one can see tests go green.
We have not had any test for TurboVNC, and then things has broken. When things break after a release we probably get project involvement in a way, but it also require maintainers to react.
I'd like to contribute the "add test" maintenance up-front to reduce the risk of needing to do work reactively later.
This project isn't new, so I'm pushy for us to get a basic test for TurboVNC at this point as we say its supported. That would make me feel more comfortable contributing to this project in reviews etc, lowering the barrier to merge etc.
@yuvipanda what do you think about #88? I'm highly motivated to resolve #68, and that can be easily by doing #88 which I think is quite easy as well - but its something that merits up-front agreement I think.
from jupyter-remote-desktop-proxy.
Yeah totally agree re: green tests making it easier to contribute. Happy to do code reviews!
If #88 is enough for you to consider #68 resolved I'm very happy for the approach described in #88 to go forward.
from jupyter-remote-desktop-proxy.
Excellent okay!
Yeah I formulated #68 as a starting point of ensuring we can start it successfully. I figure with such base environments in place, we can more easily run tests manually and then possibly also automate a few.
from jupyter-remote-desktop-proxy.
@consideRatio with #94 merged (thank you), do you think this is ready for a release or needs #93? I'm not sure when I'll have time next to push on that, as it does work locally so this is github actions debugging at this point.
from jupyter-remote-desktop-proxy.
Any other blockers to making a release?
from jupyter-remote-desktop-proxy.
I really wanted to have basic tests for turbovnc land before rather than after - with #101 I'm happy to go for a release!
from jupyter-remote-desktop-proxy.
- Blocked again by #106
from jupyter-remote-desktop-proxy.
Related Issues (20)
- Document a high level technical overview on how this project work HOT 3
- Initial websocket request can fail HOT 3
- Malfunction with a `jupyterhub-singleuser` startup influencing xsrf checks HOT 1
- Create a conda-forge feedstock for this project
- Tests failing HOT 4
- the jupyter desktop couldn't be open because of novnc js was forbidden because of 403 http error HOT 2
- Build/install fails with novnc/novnc 1.5.0 HOT 1
- What images are in scope for this project and the jupyterhub org? HOT 2
- Define VNC servers to support HOT 6
- Add a link to the hub control panel HOT 2
- Add tests to verify basic function with supported VNC servers
- TigerVNC check does not work for new versions of TigerVNC HOT 5
- Password required when running mybinder demo HOT 2
- Handle disconnects & reconnects properly HOT 1
- Add Ctrl-Alt-Del item back HOT 1
- Publish two docker images - one with TigerVNC, and one with TurboVNC? HOT 7
- Narrow python support from 3.6+ to 3.8+
- Is tigervnc-xorg-extension actually needed?
- Ensure project works against TurboVNC - currently broken? HOT 3
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from jupyter-remote-desktop-proxy.