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

Change Template source directory layout. #779

Merged
merged 2 commits into from
Aug 23, 2017

Conversation

ILMTitan
Copy link

Change template source directory layout to reduce character count below the limit when deploying to the experimental instance. Remove dependency on MSBuilder.

Note: this does not change the layout of the Template when it is created. The files deleted from GoogleAspNetCoreMvc/wwwroot/lib are parts of bower packages that are restored when a package restore is run. The files moved in GoogleAspNetWebApi are placed correctly because the TargetFolderName of the vstemplate file is changed to a path to the correct location. The names of the vstemplate files themselves only change the names of folders but don't have any other visible impact.

…ow the limit. Remove dependency on MSBuilder.
@ILMTitan ILMTitan self-assigned this Aug 17, 2017
@codecov
Copy link

codecov bot commented Aug 17, 2017

Codecov Report

Merging #779 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #779   +/-   ##
=======================================
  Coverage   10.47%   10.47%           
=======================================
  Files         496      496           
  Lines       12125    12125           
=======================================
  Hits         1270     1270           
  Misses      10855    10855

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 12eb108...b060ab5. Read the comment docs.

@ivannaranjo
Copy link
Contributor

Do the files get restored in all scenarios? What packages are we talking about here?

@ILMTitan
Copy link
Author

The packages are defined in bower.json and are bootstrap, jquery, jquery-validation and jquery-validation-unobtrusive.

A package restore occurs immediately after the template is loaded, and before every build.

Copy link
Contributor

@ivannaranjo ivannaranjo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As long as this also works when the generated project is built outside of Visual Studio I'm good to go.

@@ -74,23 +74,6 @@
<None Include="wwwroot\images\banner4.svg" />
<None Include="wwwroot\js\site.js" />
<None Include="wwwroot\js\site.min.js" />
<None Include="wwwroot\lib\bootstrap\dist\css\bootstrap-theme.css" />
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are you sure all of these files are brought back during a dotnet restore operation?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

They are not brought back by dotnet restore, but are brought back by dotnet publish, which is the command that copies the wwwroot folder.

@ILMTitan ILMTitan merged commit 99ec8f9 into GoogleCloudPlatform:master Aug 23, 2017
@ILMTitan ILMTitan deleted the shorten-templates branch August 23, 2017 23:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants