Closed weijarz closed 8 years ago
Yuck. I knew this would happen, I was trying to figure out a clean solution. I'll have to make a patch tonight.
:-(
I added Travis, and it's supposed to build the unstable branches...
I'll patch it tonight if no one provides a passing PR before then.
!
I actually can't reproduce this on my mac! (Latest versions of Atom of course)
Are you on some beta branch?
btw, relates to: https://discuss.atom.io/t/api-observe-key-binding-changes/32921
I can probably just remove those 2 lines, they were there to be safe. But regardless can't yet reproduce the issue.
Can't reproduce on a windows machine either.
Shamefully, I don't have access to a linux box at the moment, I have a branch for this I can push at any point but I'd like confirmation that it's linux that's the issue.
I noticed that Arch Linux's Atom package uses a newer electron version (1.4.3) than Atom official version (1.3.6).
https://www.archlinux.org/packages/community/x86_64/atom/ https://www.archlinux.org/packages/community/x86_64/electron/
Thanks so much. I have a quick dirty fix I could release, but I think it might cause some issues, I have an idea of how to do this a bit differently. I'll try to get a release out tonight.
Lee Dohm brings up that the build you're on is not supported.
That said this was on my list to fix for a while. @johngeorgewright Mind taking a quick peek at this and signing off? I'll have another friend give it a peek too.
Also, I set up daily builds with travis ci for jumpy + qolor now. Should help, if and when the beta branch breaks.
Soz... on London time, just saw this. Pull request looks good.
[Enter steps to reproduce below:]
shift+enter
Atom Version: 1.10.2 Electron Version: 1.4.2 System: linux 4.4.23-1-lts Thrown From: jumpy package, v3.1.2
Stack Trace
Failed to activate the jumpy package
Commands
Config
Installed Packages