-
Notifications
You must be signed in to change notification settings - Fork 22
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
Update ASP.NET 5 deployment script to publish with --no-source and --configuration Release #28
Comments
Should we make this configurable, so that users can go either way? |
Sounds good. What does that mean exactly? An env var to control? Portal option? |
Well, it could be either, depending on whether it's an edge case of something common. Interestingly, it looks like @ahmelsayed already added a switch. Looking at this line, if you set a Not saying this is necessarily good enough, but it should at least allow you to test the scenario with the other flags, to validate that you get the expected improvements and that everything works. Can you try setting |
I just set SCM_DNU_PUBLISH_OPTIONS to --configuration Release but after deploying to Azure via github my app is broken as long as I have #if DEBUG in my Startup.cs |
Side question: Why are you switching on debug instead of the environment? |
Because I didn't know any better :) |
This is still breaking my site (rc1-final) when publishing to Azure:
System.IO.FileNotFoundException |
Make Kudu deployment script publish with
--no-source --configuration Release
so app doesn’t have to compile on startup and is compiled with optimizations.The text was updated successfully, but these errors were encountered: