Skip to content

Conversation

@abonie
Copy link
Member

@abonie abonie commented Jun 16, 2025

Create a solution file to be run by copilot on a non-windows machine. Exclude Editor project from it due to windows dependencies. The LSP project itself targets net8.0-windows - TODO investigate if this is needed, we have to either conditionally change the target framework for copilot or just drop "-windows" part

Create a solution file to be run by copilot on a non-windows machine.
Exclude Editor project from it due to windows dependencies. The LSP
project itself targets net8.0-windows - TODO investigate if this is
needed, we have to either conditionally change the target framework for
copilot or just drop "-windows" part
@github-actions
Copy link
Contributor

✅ No release notes required

Point is to enable agents running on ubuntu to build - might not work
properly on non-windows though.
@abonie abonie marked this pull request as ready for review June 16, 2025 20:09
@abonie abonie requested a review from a team as a code owner June 16, 2025 20:09
@github-project-automation github-project-automation bot moved this from New to In Progress in F# Compiler and Tooling Jun 17, 2025
@abonie abonie merged commit e22f251 into dotnet:main Jun 17, 2025
39 checks passed
T-Gro added a commit that referenced this pull request Jun 23, 2025
* Add new sln file for LSP proj on ubuntu

Create a solution file to be run by copilot on a non-windows machine.
Exclude Editor project from it due to windows dependencies. The LSP
project itself targets net8.0-windows - TODO investigate if this is
needed, we have to either conditionally change the target framework for
copilot or just drop "-windows" part

* Change target on non-windows platform

Point is to enable agents running on ubuntu to build - might not work
properly on non-windows though.

---------

Co-authored-by: Adam Boniecki <20281641+abonie@users.noreply.github.com>
Co-authored-by: Adam Boniecki <abonie@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants