Skip to content
This repository was archived by the owner on Apr 8, 2020. It is now read-only.

Target dotnet TFM's for class libraries#23

Merged
SteveSandersonMS merged 3 commits into
aspnet:masterfrom
henkmollema:projectjson
Dec 14, 2015
Merged

Target dotnet TFM's for class libraries#23
SteveSandersonMS merged 3 commits into
aspnet:masterfrom
henkmollema:projectjson

Conversation

@henkmollema

Copy link
Copy Markdown
Contributor
  • Target net451 and dotnet5.4
  • Add SPA services project to solution (with .xproj)
  • Update .gitignore
  • Add Git repo information to project.json

- Target `net451` and `dotnet5.4`
- Add SPA services project to solution (with .xproj)
- Add NuGet.config
- Update .gitignore
@dnfclas

dnfclas commented Dec 10, 2015

Copy link
Copy Markdown

Hi @henkmollema, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!

The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.

TTYL, DNFBOT;

Comment thread .gitignore Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

About this file - it would be nice to keep it minimal. Could we limit these entries just to the files that appear without the developer's involvement on the two most common dev setups: VS and VS Code?

@henkmollema

Copy link
Copy Markdown
Contributor Author

@SteveSandersonMS updated

SteveSandersonMS added a commit that referenced this pull request Dec 14, 2015
Target dotnet TFM's for class libraries
@SteveSandersonMS SteveSandersonMS merged commit 75b959d into aspnet:master Dec 14, 2015
@SteveSandersonMS

Copy link
Copy Markdown
Member

Thanks! It's in.

@henkmollema

Copy link
Copy Markdown
Contributor Author

Cool! 😄

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.

3 participants