Skip to content

Reference the minified js files from templates #11700 #12014

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

Merged
merged 2 commits into from
Jul 10, 2019
Merged

Reference the minified js files from templates #11700 #12014

merged 2 commits into from
Jul 10, 2019

Conversation

isaacrlevin
Copy link
Contributor

Summary of the changes

  • reference minified js in templates

Addresses #11700

isaacrlevin and others added 2 commits June 26, 2019 16:47
merge

Co-authored-by: David Fowler <[email protected]>
Co-authored-by: Pranav K <[email protected]>
Co-authored-by: Ryan Nowak <[email protected]>
Co-authored-by: Justin Kotalik <[email protected]>
Co-authored-by: Kristian Hellang <[email protected]>
Co-authored-by: Ajay Bhargav Baaskaran <[email protected]>
Co-authored-by: null <42748379+dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: Hao Kung <[email protected]>
Co-authored-by: Doug Bunting <[email protected]>
Co-authored-by: Chris Ross <[email protected]>
Co-authored-by: Thomas Levesque <[email protected]>
@isaacrlevin
Copy link
Contributor Author

@ryanbrandenburg failed flaky tests

@ryanbrandenburg
Copy link
Contributor

The helix tests are not actually required to pass in order to merge. That said these failed to run, so I'd like to wait and make sure they don't surface anything relevant to this PR now that I've retried them.

@Pilchie Pilchie added the area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates label Jul 10, 2019
@ryanbrandenburg
Copy link
Contributor

Oh, apparently there's a helix outage/problem at the moment, so those are unlikely to succeed. AspNetCore-ci covers the templating tests anyway, we should be good to merge.

@ryanbrandenburg ryanbrandenburg merged commit 6b421fe into dotnet:master Jul 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants