Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add string extension IsHtmlFile #26

Merged
merged 3 commits into from
Apr 19, 2016
Merged

Add string extension IsHtmlFile #26

merged 3 commits into from
Apr 19, 2016

Conversation

heldersepu
Copy link
Contributor

This is to address bug #25

- Move all the minification to an extension to StreamReader
- Get rid of global var _features
- Extend the Features class with a constructor and add MaxLength
@deanhume
Copy link
Owner

Great stuff - thanks for this!

@deanhume deanhume merged commit b6a72d8 into deanhume:master Apr 19, 2016
@heldersepu
Copy link
Contributor Author

I'm glad you like it...

Looking at the way we retrieve the files, I think is a bit too complex we
might be able to simplify it
The Directory.GetFiles has an overload that allows a search pattern and we
can also add the AllDirectories searchOption...

https://msdn.microsoft.com/en-us/library/ms143316(v=vs.110).aspx

But that is just a thought I did not research any of it

On Tue, Apr 19, 2016 at 12:11 PM, Dean notifications@github.com wrote:

Great stuff - thanks for this!


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#26 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants