Non-existant file still gets its success callback called

The following prints out “shouldnthappen”

$script('doesntExist.js', function() {
  console.log('shouldnthappen')
}, function(notFound) {
    console.log(notFound)
})

This makes me think script.js isn’t in any way production ready.

[ded]

this is a valid concern but with all respect leave your sarcasm elsewhere. your request to change core is an opinion of which has been purposely decided in the library to load non-existent or errored files. you would otherwise never receive a callback and may mistakenly attempt to wait for an event that may never occur. it’s up to the developer to make sure their scripts load properly.

Eg:

$('cdn.jquery.com/latest.js', function () {
  if (typeof $ !== 'undefined') ready()
  else $script('/js/jquery.js', ready)
})

[kzap]

@amorey the behaviour is fine, the documentation could be more verbose 😃