Added directory support #32
Conversation
Added support for common version-number-naming-conventions in filenames
| get | ||
| { | ||
| return Files; | ||
| } |
There was a problem hiding this comment.
Should this return Files.Contat(Directories)? A more general question - does this PR work with nested directories (I don't think you explicitly mentioned it, but I notice the test folder is a single directory)?
There was a problem hiding this comment.
Yes it should, but I did not implement sub-directories, so I left it out.
There was a problem hiding this comment.
Are subdirectories unnecessary for ASP.NET bundling, or will embedded resource bundling only work with single subdirectories?
Sorry for dumb questions but I don't use Bundles that much, and if there's any unexpected behaviour I'll need to document it to reduce the number of people raising Github Issues.
| @@ -7,7 +7,7 @@ | |||
| <title></title> | |||
| <script src="/alert.js" type="text/javascript"></script> | |||
| <script src="/alert2.js" type="text/javascript"></script> | |||
| <script src="/Scripts/alert.3.js" type="text/javascript"></script> | |||
| <script src="/Scripts/alert-1.12.34567.min.js" type="text/javascript"></script> | |||
| <script src="/3rd3Party/3select2-3.4.5/select2.js" type="text/javascript"></script> | |||
| <script src="/3rd3Party/hammer.js-1.0.10/hammer.js" type="text/javascript"></script> | |||
There was a problem hiding this comment.
/3rd3Party/3select2-3.4.5/select2.js looks pretty tricky - does it work ok with the regex? (sorry I don't have access to VS to test this stuff out)
No description provided.