Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
This repository was archived by the owner on Apr 20, 2023. It is now read-only.

Align f# web templates (preview2) - #4985

#4985
Merged
TheRealPiotrP merged 1 commit into
dotnet:rel/1.0.0-preview2dotnet/cli:rel/1.0.0-preview2from
enricosada:align_fsharp_templates_p2_webenricosada/cli:align_fsharp_templates_p2_webCopy head branch name to clipboard
Dec 10, 2016
Merged

Align f# web templates (preview2)#4985
TheRealPiotrP merged 1 commit into
dotnet:rel/1.0.0-preview2dotnet/cli:rel/1.0.0-preview2from
enricosada:align_fsharp_templates_p2_webenricosada/cli:align_fsharp_templates_p2_webCopy head branch name to clipboard

Conversation

@enricosada

Copy link
Copy Markdown

aligned F# web templates after PR #4980 changed the c# one.
just package version changes

Affect preview2

@piotrpMSFT @MattGertz @livarcocc for approval for merge in next preview2 🙏 😄

@MattGertz

Copy link
Copy Markdown

@enricosada Please tag me again once @piotrpMSFT @livarcocc have signed off (I need to be the final signoff, and I have inbox rules that filter everything out unless I am specifically tagged again.)

All -- please note that this would be for RC3 alignment not RC2 alignment -- RC2 is now at "recall class."

@TheRealPiotrP TheRealPiotrP left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This looks good to me, but I'd actually like @mlorbetske to sign off as I'm not familiar with ASP.NET dependencies.

@enricosada

Copy link
Copy Markdown
Author

Adding the online diff If needed http://www.mergely.com/ESFER5l6/

@TheRealPiotrP

Copy link
Copy Markdown

@MattGertz

@TheRealPiotrP
TheRealPiotrP merged commit 7dbb4a3 into dotnet:rel/1.0.0-preview2 Dec 10, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.