Skip to content

Conversation

@MiYanni
Copy link
Member

@MiYanni MiYanni commented Nov 15, 2023

Reverts: #1983

Summary

This is a coordinated update for dotnet/command-line-api in the repos that DARC flows to. There needs to be only 1 version of this dependency for Source Build to function properly. See: dotnet/sdk#33126

@MiYanni
Copy link
Member Author

MiYanni commented Nov 17, 2023

CC @jjonescz @MichaelSimons

(I would have requested as reviewers, but I don't seem to have the capability to do that in this repo)

@jjonescz
Copy link
Member

Can I merge this or does it need to be coordinated somehow?

@MiYanni
Copy link
Member Author

MiYanni commented Nov 20, 2023

@jjonescz We were waiting for the Roslyn integration test issue to be resolved. Plus, runtime had merged theirs ahead of time and I had to revert that. So, I gotta revert that revert once Roslyn is ready. Tracking in this comment: dotnet/sdk#33126 (comment)

@MiYanni
Copy link
Member Author

MiYanni commented Nov 23, 2023

@jjonescz This now can be merged immediately. The SDK PR and Runtime PR have been merged.

@jjonescz jjonescz merged commit f0e966d into dotnet:main Nov 23, 2023
@ghost ghost added this to the Next milestone Nov 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants