Skip to content
This repository has been archived by the owner on Oct 12, 2022. It is now read-only.

ASP.NET Core quick start #203

Merged
merged 11 commits into from
Apr 7, 2016
Merged

ASP.NET Core quick start #203

merged 11 commits into from
Apr 7, 2016

Conversation

sandersn
Copy link
Member

This branch is a fork of the asp-net-quick-start branch for ASP.NET 4. I'll wait to merge this one to master until the ASP.NET 4 one is in.


2. Use PackageInstaller to install Angular 2, systemjs and Typings.

![Use PackageInstaller to install angular2](packageinstaller-angular2.png)

Choose a reason for hiding this comment

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

you'd also likely need rxjs here

Copy link
Member Author

Choose a reason for hiding this comment

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

It comes in as a dependency of angular and later I have MSBuild copy Rx.js along with angular2.js. So I think it's already there.

@mhegazy
Copy link
Contributor

mhegazy commented Mar 24, 2016

@sayedihashimi and @danroth27, can you give this sample a quick review?

sandersn and others added 4 commits March 25, 2016 09:22
Thanks @robmorwald for the help.
1. Mark "things you click" in **bold**.
2. Mark a few more "things you type" in `monospace`.
3. Adjust list-item spacing to be legal kramdown.
@sandersn
Copy link
Member Author

@DanielRosenwasser asp-net-5-quick-start branches off of asp-net-quick-start. It's not meant to merge from master until asp-net-quick-start is merged.

@DanielRosenwasser
Copy link
Member

I'm confused about this branch. Is this ASP.NET 4? Is it ASP.NET 5? Why is it out for review exactly?

@sandersn
Copy link
Member Author

It's 5 branched from 4. I was going to put 4 and 5 in the same file and later decided not to.

@sandersn sandersn changed the title ASP.NET 5 quick start ASP.NET Core quick start Apr 1, 2016
@mhegazy
Copy link
Contributor

mhegazy commented Apr 4, 2016

👍


5. Choose **ASP.NET 5 Empty**

I unchecked "Host in the cloud" since this will be a local demo.
Copy link
Member

Choose a reason for hiding this comment

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

Don't refer to "I", but use a guiding collective voice (or whatever this is called).

For instance:

Let's uncheck "Host in the cloud" since we're going to run this locally.

Copy link
Member Author

Choose a reason for hiding this comment

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

done

@mhegazy
Copy link
Contributor

mhegazy commented Apr 7, 2016

👍

@sandersn sandersn merged commit fa9e2be into master Apr 7, 2016
@sandersn sandersn deleted the asp-net-5-quick-start branch April 7, 2016 21:48
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.

None yet

4 participants