RequireJS Working Example?

[plusjade]

I'm struggling to get this working with requireJS. I've tried the examples in the README with no luck.

Here is my control case:

define([
  'require',
  'jquery',
  'mustache',
  'js-yaml.min',
], function(require, $){

  var JB = { 
    init : function(boot){
      var doc = jsyaml.load('greeting: hello\nname: world');
      console.log(doc); // this works fine.

      //require('js-yaml.min'); // this works both uncommented and commented.

      var singleDoc = require('http://127.0.0.1/~jade/javascripts/require-js/tags.yaml').shift();
      console.log(singleDoc); 
      // This does not work
      // Uncaught Error: Module name 'http://127.0.0.1/~jade/javascripts/require-js/tags.yaml' has not been loaded yet for context: _
      // http://requirejs.org/docs/errors.html#notloaded

      if(typeof boot === "function") boot();
    } 
  };

  return JB;
});

As per the error documentation (http://requirejs.org/docs/errors.html#notloaded) I have tried to load the tags.yaml first in the define statement. The tags.yaml loads fine but since its not javascript, It returns a syntax error.

Any help would be appreciated, thanks.

[ixti]

Hm... It looks like you're right. In fact I'm not really familiar with RequireJS, so I simply copy-pasted jQuery's define block:

if ( typeof define === "function" && define.amd) {
  define( "jsyaml", [], function () { return jsyaml; } );
}

In terms of what you want to do, as far as I can see, it's not enough. If you'll provide a necessary code that RequireJS needs to make JsYAML 100% compatible with RequireJS, I'll be proud to merge it.

[plusjade]

Ok thank you. I am still working on this. When I get it figured out nicely I will send my pull request!

[ixti]

We have removed AMD definition in the trunk, as it's obviously not ready yet. Please, if you are confident with RequireJS and can provide a pull-request - I will be happy to merge it in.