razor Publish CLaSP to VS Mac... maybe?

Publish CLaSP to VS Mac... maybe?

Open davidwengier opened this issue 1 year ago • 1 comments

We should probably ship this DLL in VS Mac. Might be useful.

Oct 19 '22 04:10 davidwengier

Hello @davidwengier!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (`@msftbot`) and give me an instruction to get started! Learn more here.

Oct 19 '22 05:10 ghost

CLaSP ships out of Razor, not Roslyn?

Oct 19 '22 16:10 sandyarmstrong

It builds out of Roslyn, but isn't in the standard package AFAIK.. but actually, now that you mention it, the compat checker presumably would have complained about Roslyn's use of it, if the DLL wasn't present. Maybe the fix is elsewhere on the vsmac side...

Oct 19 '22 19:10 davidwengier

Okay, don't need this. Referencing CLaSP from vsmac directly works fine, and is presumably needed for C# LSP.

Oct 19 '22 23:10 davidwengier

razor razor copied to clipboard

Publish CLaSP to VS Mac... maybe?

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

razor
razor copied to clipboard

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (`@msftbot`) and give me an instruction to get started! Learn more here.