silent failure is uncool

bertramdev / asset-pipeline

The core implementation of the asset pipeline for the jvm

193 stars 92 forks source link

Closed ScottIngram closed 6 years ago

ScottIngram commented 7 years ago

//= require this-file-does-not-exist

will silently fail to include this-file-does-not-exist.js -- it should at the very least log.debug("some message")

davydotcom commented 6 years ago

not sure what version you are on but check out DirectiveProcessor#requireFileDirective() LINE 281 There is a log warning for this.

code sample

chrisbrookes commented 6 years ago

I guess it's not completely silent, but it doesn't fail the Gradle build. An option to fail the build in this case might be a good idea?