These are chat archives for opal/opal

4th
Jun 2015
Vais Salikhov
@vais
Jun 04 2015 11:54 UTC
@elia any idea what happened with is_phantom method?
mspec_phantom is failing on master for me :(
Elia Schito
@elia
Jun 04 2015 12:13 UTC
was/is a typo
I rwote is_phatnom or something like htat
@vais should be fixed on master tho
Vais Salikhov
@vais
Jun 04 2015 12:20 UTC
Sweet, rebasing with the latest master fixed it. As always, thank you, @elia!
Elia Schito
@elia
Jun 04 2015 12:20 UTC
typos are alwasy lovley :heart:
Vais Salikhov
@vais
Jun 04 2015 12:21 UTC
Speaking of which, I'm almost done with my jshint-a-palooza :sweat_smile:
There's a couple more compiler fixes in there to take care of some edge cases with for-in loop that came up when jshinting our entire spec suite. That's a lot of ruby and a lot of js, a real good smoke test for the compiler!
Elia Schito
@elia
Jun 04 2015 12:25 UTC
:top:
Vais Salikhov
@vais
Jun 04 2015 12:41 UTC
@elia travis is driving me nuts. Is it possible it is not grabbing the latest commit for the spec/corelib submodule somehow?
Elia Schito
@elia
Jun 04 2015 12:42 UTC
seems unlikely given how subm…
wait,
try removing spec/corelib from cached paths
@vais
Vais Salikhov
@vais
Jun 04 2015 12:43 UTC
Where would one manage cached paths in travis?
Settings > Caches I just see Pushes and Pull Requests :confused:
Elia Schito
@elia
Jun 04 2015 12:44 UTC
sorry I mean .travis.yml
Vais Salikhov
@vais
Jun 04 2015 12:46 UTC
Sorry, @elia, you mean, like, delete that line from .travis.yml and push another commit?
Elia Schito
@elia
Jun 04 2015 12:46 UTC
yep
was an attempt I made at speeding up the build
Vais Salikhov
@vais
Jun 04 2015 12:48 UTC
Ok, keeping my fingers crossed :)
Vais Salikhov
@vais
Jun 04 2015 12:59 UTC
Nice, that did it, @elia, you da man, I never would have guessed that was the problem!
opal/opal#903
"Run JSHint as part of Opal test suite"
a.k.a jshint-a-palooza is finished :shipit:
Really could use @adambeynon's expertise reviewing vais/opal@5da42a7, vais/opal@a5282a4, and vais/opal@843968d though. Holding by breath :sweat_smile:
Jared White
@jaredcwhite
Jun 04 2015 14:09 UTC
Hi folks - just a reminder that next week is Opalist week (Issue #5). :) cc @fkchang
Adam Jahn
@ajjahn
Jun 04 2015 14:11 UTC
Hey everyone, I've been interested in using opal on a project that requires a few jQuery plugins. I assume I'll need to wrap the plugins in someway (using native, or a wrapper class/module). It's seems straight forward looking at opal-jquery's source how it's wrapping jQuery, but for some reason I'm not grasping how to do it with a plugin. Can anyone point me in the right direction? Or suggest some existing jQuery plugin wrappers at which I could look?
Adam Beynon
@adambeynon
Jun 04 2015 15:37 UTC
@ajjahn if it is a simple method, which takes simple parameters you can use the expose method
e.g., with bootstrap’s modal() plugin:
Element.expose :modal
that assumes we can pass parameters which can be handled directly, e.g. strings, numbers etc
or no args at all
Adam Jahn
@ajjahn
Jun 04 2015 15:53 UTC
@adambeynon Intersting; I'll have a look at that!
One road I traveled down for more complex methods was to create a module that implements the plugin's method(s) by shelling out to JS and then including it in the Element class: Element.send(:include, PluginModule). Works, but I wouldn't know if there is a better way.
BrandonMathis
@BrandonMathis
Jun 04 2015 17:33 UTC
@RickCarlino We promoted one of our devs to a Volt developer
Rick Carlino
@RickCarlino
Jun 04 2015 20:24 UTC
@BrandonMathis I saw that. What a valiant developer indeed.
Ryan Stout
@ryanstout
Jun 04 2015 21:56 UTC
@jaredcwhite when does the next opalist come out? (The next volt release and a blog post about the new stuff is almost ready - if your interested in including it :-)
Ryan Stout
@ryanstout
Jun 04 2015 22:33 UTC
is there a good way to check if I have an opal object from opal? (as opposed to just a JS object)
Jeremy Evans
@jeremyevans
Jun 04 2015 22:37 UTC
Ugly: object.object_id rescue (it's a JS object)
ryanstout, if it has $$class it's an opal object
but it will obviously fail on null and undefined
in theory BasicObject === object would work in all cases
Ryan Stout
@ryanstout
Jun 04 2015 22:55 UTC
@meh cool, even for null and undefined?
Jeremy Evans
@jeremyevans
Jun 04 2015 22:57 UTC
@ryanstout Yes, that appears to work for both null and undefined
Ryan Stout
@ryanstout
Jun 04 2015 23:02 UTC
@jeremyevans cool, thanks (guess I could have checked :-)
ryanstout, the last one should be