Skip to content

Add reference assembly generations support#1093

Merged
pakrym merged 22 commits intomasterfrom
pakrym/refs
Feb 11, 2019
Merged

Add reference assembly generations support#1093
pakrym merged 22 commits intomasterfrom
pakrym/refs

Conversation

@pakrym
Copy link

@pakrym pakrym commented Feb 8, 2019

Adds a way to generate reference assembly projects

TODO:

  • Doc comment xmls
  • ApiCompat checks

@pakrym pakrym requested a review from natemcmaster February 8, 2019 22:51
@pakrym pakrym requested a review from Tratcher as a code owner February 8, 2019 22:51
@Tratcher
Copy link
Member

Tratcher commented Feb 8, 2019

Wow, please tell me there's a test/build task that verifies these are up to date?

@pakrym
Copy link
Author

pakrym commented Feb 8, 2019

I'll add it to code check.

@pakrym
Copy link
Author

pakrym commented Feb 9, 2019

@natemcmaster
Any idea why this code check only fired on this PR:

2019-02-09T00:10:17.1842958Z error : Missing project. Solution references a project which does not exist: D:\a\1\s\src\JSInterop\Microsoft.JSInterop.JS\src\Microsoft.JSInterop.JS.csproj. [D:\a\1\s\Extensions.sln] 

File was removed in #1048.

@natemcmaster
Copy link

Looks like we were getting a false-positive test pass on codecheck due to global.json issues

image

Copy link

@natemcmaster natemcmaster left a comment

Choose a reason for hiding this comment

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

Looks like a great start. Still reviewing, but I see you're still pushing changes, so i'll pause here for now.

natemcmaster and others added 8 commits February 8, 2019 16:52
…xtensions.DependencyInjection.Specification.Tests.csproj

Co-Authored-By: pakrym <pavel@krymets.com>
Co-Authored-By: pakrym <pavel@krymets.com>
…soft.Internal.Extensions.Refs.csproj

Co-Authored-By: pakrym <pavel@krymets.com>
@pakrym
Copy link
Author

pakrym commented Feb 9, 2019

@natemcmaster I'm done with changes for today.

@pakrym
Copy link
Author

pakrym commented Feb 11, 2019

🆙📅

@pakrym pakrym changed the title [WIP] Add reference assembly generations support Add reference assembly generations support Feb 11, 2019
@pakrym pakrym merged commit 8ad1395 into master Feb 11, 2019
@natemcmaster natemcmaster deleted the pakrym/refs branch May 3, 2019 06:35
JunTaoLuo pushed a commit to dotnet/aspnetcore that referenced this pull request Feb 12, 2020
JunTaoLuo pushed a commit to dotnet/aspnetcore that referenced this pull request Feb 15, 2020
maryamariyan pushed a commit to maryamariyan/runtime that referenced this pull request Feb 28, 2020
maryamariyan pushed a commit to maryamariyan/runtime that referenced this pull request Mar 2, 2020
maryamariyan pushed a commit to maryamariyan/runtime that referenced this pull request Mar 11, 2020
maryamariyan pushed a commit to maryamariyan/runtime that referenced this pull request Mar 11, 2020
maryamariyan pushed a commit to maryamariyan/runtime that referenced this pull request Mar 17, 2020
maryamariyan pushed a commit to maryamariyan/runtime that referenced this pull request Mar 27, 2020
JunTaoLuo pushed a commit to dotnet/aspnetcore that referenced this pull request Nov 17, 2020
@ghost ghost locked as resolved and limited conversation to collaborators May 29, 2023
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