Skip to content
This repository was archived by the owner on Feb 13, 2019. It is now read-only.

Conversation

@sesispla
Copy link
Contributor

Fixes #685.

Summary of the changes in this PR:

  • Modified Dockerfile.txt template to fit dotnet cli commands (restore, build...)
  • Added Microsoft.Extensions.Configuration.CommandLine NuGet package to project.json web, webapi and web basic templates
  • Added CommandLine ConfigurationBuilder to fix Kestrel endpoint configuration through Dockerfile ENTRYPOINT

Thanks!

/cc
@OmniSharp/generator-aspnet-team-push

sayedihashimi and others added 11 commits May 16, 2016 14:19
See aspnet/Templates#575 for details.
This commit introduces fix from aspnet/Templates
to generator content

Thanks!
This commit adds Add Microsoft.Extensions.Options.ConfigurationExtensions
to default projects packages as in:
aspnet/Templates#471

Thanks!
* Introduce Bundler to web basic (StarterWeb) template

This commit brings changes from aspnet/Templates#592
to starter web project template. The Gulp, Grunt options
are removed with NPM support removal.

Thanks!

* ⬆️ Update Bundler version

The 2.0.208 fixes *nix paths problem

* Add missing Bundle to Semantic UI overrides

* Set post-install of NPM to false

* Fix output path names
This commit brings changes from aspnet/Templates#592
to web project template. The Gulp, Grunt options
are removed with NPM support removal.

Thanks!
Afte adding Bundler support from aspnet/Templates#592
the --grunt option has no purpose
@peterblazejewicz
Copy link
Member

@sesispla
It looks like there was a problem when submitting your commit/PR. Please start from master branch. The rc2 is historic branch. I'll close RC2 related topics or rename them to post-rc2 to avoid confusion. RC2 is the current release or master branch.
Thanks!

This commit adds Bundler command to script section of the project file
See: aspnet/Templates@8b42fc6

Thanks!
This commit adds custom css rule to site.css
See: aspnet/Templates@bc93039

Thanks!
@peterblazejewicz
Copy link
Member

closing in favour of: #743

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants