Skip to content

Commit

Permalink
fixing cjs extensions
Browse files Browse the repository at this point in the history
  • Loading branch information
davydotcom committed Apr 11, 2022
1 parent fee46a7 commit b02d327
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ import groovy.transform.CompileStatic
@CompileStatic
class JsAssetFile extends AbstractAssetFile {
static final List<String> contentType = ['application/javascript', 'application/x-javascript','text/javascript']
static List<String> extensions = ['js']
static List<String> extensions = ['js','cjs']
static String compiledExtension = 'js'
static processors = [JsProcessor,JsNodeInjectProcessor,BabelJsProcessor,JsRequireProcessor]
Pattern directivePattern = ~/(?m)^\/\/=(.*)/
Expand Down
2 changes: 2 additions & 0 deletions asset-pipeline-docs/src/asciidoc/introduction.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,8 @@ This guide documents configuration and usage examples of the asset-pipeline plug

=== Release History

* April 10, 2022
** - 3.4.6 - .cjs Extension now considered a valid javascript extension

This comment has been minimized.

Copy link
@darxriggs

darxriggs Apr 26, 2022

I guess this is intended for hosting on http://www.asset-pipeline.com/manual/#introduction - right?
The website currently only contains the information up to 3.0.10. So it seems to require an update.
There is also #291 about this.

This comment has been minimized.

Copy link
@darxriggs
* April 8, 2022
** - 3.4.5 - package.json detection was happening post looking for index.js Causing confusion between ES2017 modules and CJS
* March 22, 2022
Expand Down
2 changes: 1 addition & 1 deletion gradle.properties
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
version=3.4.5
version=3.4.6
org.gradle.warning.mode=none

0 comments on commit b02d327

Please sign in to comment.