Remove Spa-templates from repo #49413
Conversation
wtgodbe
requested review from
a team,
captainsafia and
mkArtakMSFT
as code owners
ghost
added
area-mvc
| @@ -35,15 +35,6 @@ internal static class TemplatePackageInstaller | |||
| "Microsoft.DotNet.Web.ProjectTemplates.6.0", | |||
| "Microsoft.DotNet.Web.ProjectTemplates.7.0", | |||
| "Microsoft.DotNet.Web.ProjectTemplates.8.0", | |||
| "Microsoft.DotNet.Web.Spa.ProjectTemplates.2.1", | |||
There was a problem hiding this comment.
@DamianEdwards do we want to leave around everything except 8.0?
There was a problem hiding this comment.
@danroth27 your call. I would think we would remove all of these so that the 8.0 SDK doesn't carry any SPA project templates at all.
There was a problem hiding this comment.
I think we should remove them all.
|
Thanks @wtgodbe! |
|
Do you know where are the new templates after removing the old ones? |
|
Hi @zcxsythenew. It looks like you just commented on a closed PR. The team will most probably miss it. If you'd like to bring something important up to their attention, consider filing a new issue and add enough details to build context. |
Hi @zcxsythenew. The new ASP.NET Core SPA templates now ship as part of Visual Studio. You can try them out with the latest Visual Studio preview release. |
|
Hi @danroth27. It looks like you just commented on a closed PR. The team will most probably miss it. If you'd like to bring something important up to their attention, consider filing a new issue and add enough details to build context. |
@danroth27 Yes, I know it, but there is something wrong with these templates, and I want to raise an issue or PR. Are they not available on GitHub? (See #51341 for details) |
|
Hi @zcxsythenew. It looks like you just commented on a closed PR. The team will most probably miss it. If you'd like to bring something important up to their attention, consider filing a new issue and add enough details to build context. |
Resolves #49388