-
Notifications
You must be signed in to change notification settings - Fork 3.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
(fix) display tooltips on All demos view #4702
Conversation
@emackey do you want to look at this? |
@tekhaus we updated electron in master, so if you update your branch CI should pass |
@hpinkos merged from upstream master and pushed to my fork... how do I get CI to rerun on this PR ? |
merge your fork's master into this branch and push that up. That should make CI run again |
Thanks @tekhaus, I can confirm this fixes the problem. There were some merge issues with your branch, so I just merged master and pushed that up. |
Oh actually, before I merge this, can you add yourself to https://github.com/AnalyticalGraphicsInc/cesium/blob/master/CONTRIBUTORS.md? |
Also, one last small request: please update CHANGES.md. |
@pjcozzi I'll update CHANGES in master |
#4454