Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Implement as a RequireJS module if available

See original GitHub issue

Rather than forcing the namespace this would let it work properly with modern good practices - there’s various patterns, but basically if there’s no define function defined then nothing changes, otherwise it should effectively have define(function(){return ADL;}) - but possibly without using the global namespace at all.

Issue Analytics

State:
Created 8 years ago
Comments:6 (2 by maintainers)

Top GitHub Comments

1reaction

zpetterdcommented, Mar 8, 2017

Kind of… I ended up having to rewrite most of the structure to mean it can work in both a browser and Node JS environment alike. In the end I just used https://github.com/RusticiSoftware/TinCanJS as it works right out of the box

On Thu, 9 Mar 2017, 5:57 AM Lou Wolford notifications@github.com wrote:

@zapur1 https://github.com/zapur1 : Have there been any updates to a completed node module with tests?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/adlnet/xAPIWrapper/issues/60#issuecomment-285133634, or mute the thread https://github.com/notifications/unsubscribe-auth/ABzk9qrrn0JyLuXXhA71W0usfnS1HxqSks5rjvoXgaJpZM4HD6lO .

0reactions

ljwolfordcommented, Mar 8, 2017

@zapur1 : Have there been any updates to a completed node module with tests?