Thanks for these
Make a separate issue for the unfixed bits so we can address separately then I can merge the rest

I can create issues for the unfixed issues of course. But the unit-test will still fail, since it is a generic one that detects all five issues.
The code in this PR would remain unchanged then.
Is this ok?

About the failing check:
Restoring the packages works perfectly fine locally. I don't know how to fix it in AppVeyor.

Thank you Max for fixing the build. Now it's only the failing unit-test that makes the checks fail.
This is due to the open issues mentioned above (yield, async and ES6.js). These were in the project before this PR and are covered with a unit-test now.

I can probably fix the async thing later, I'll add it to this pr

Is something missing in our request? Do you have a timeframe for the next release?

I'll take a look later, apologies

When I come to run the tests for this I get:

System.TypeLoadException : Cannot load ClearScript V8 library. Load failure information for ClearScriptV8.win-x64.dll:
D:\git\NUglify\src\NUglify.Tests\bin\Debug\runtimes\win-x64\native\ClearScriptV8.win-x64.dll: The specified module could not be found
D:\git\NUglify\src\NUglify.Tests\bin\Debug\ClearScriptV8.win-x64.dll: The specified module could not be found

Nuget Restore doesn't fix this, is this right? Am I missing something?

When I come to run the tests for this I get:

System.TypeLoadException : Cannot load ClearScript V8 library. Load failure information for ClearScriptV8.win-x64.dll:
D:\git\NUglify\src\NUglify.Tests\bin\Debug\runtimes\win-x64\native\ClearScriptV8.win-x64.dll: The specified module could not be found
D:\git\NUglify\src\NUglify.Tests\bin\Debug\ClearScriptV8.win-x64.dll: The specified module could not be found

Nuget Restore doesn't fix this, is this right? Am I missing something?

Some references in Nuglify.Tests.nuget.props depends from windows user profile and nugget version. This changes are not pushed.
Can u try to install missing packages manually? At least the paths stay the same.

• Microsoft.ClearScript
• Microsoft.ClearScript.V8

Please try it again with the new commit. It contains a new file NUglify.Tests.nuget.props.

So, the tests fail, as per AppVeyor

Which tests fail? We only fixed 2 of 4 problems for linebreaks, now the 2 others run into an error. The other failing tests we dont touched.

Just to avoid misunderstandings:
We added a unit-test that tests the js-minification with the most line-breaks possible. It does use all existing test-files for that.
Doing this, we found 4 already existing issues concerning the line-breaks.
Two of them were bothering us. And we got lucky: We were able to fix both of them. For the other 2, we lack the necessary knowledge. Thus these remain unfixed. But we left the unit-test failing to show the remaining issues.

TLDR:
We fixed two issues in NUglify and added a unit test that detects two more. We do not have the knowledge necessary to fix the other two issues. Thus we still improved the project a tiny bit without making anything else worse.