Skip to content

[repo-assist] Support .slnx solutions in paket restore (#4443) - #4452

Merged
fpellet merged 2 commits into
masterfrom
repo-assist/fix-issue-4443-slnx-restore-ee71a070001dd7c1
Oct 17, 2026
Merged

fpellet merged 2 commits into
masterfrom
repo-assist/fix-issue-4443-slnx-restore-ee71a070001dd7c1

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

🤖 This is an automated PR from Repo Assist.

Closes #4443

Root cause: The post-restore step in Utils.fs (which calls dotnet restore on solutions) only looked for *.sln, so .slnx solutions were ignored and it fell back to project files.

Fix: Also pick up *.slnx in the root folder. Release note added.

Scope/trade-off: This covers only the dotnet restore invocation; NugetConvert still reads .sln only (the .slnx XML format would need a separate parser).

Test Status

  • Build not run: dotnet build fails in this sandbox because dotnet tool restore / paket restore needs network access (infrastructure). The change is a small, type-safe array composition. No unit test added (the function shells out to dotnet).

Generated by 🌈 Repo Assist, see workflow run. Learn more.
Comment /repo-assist to run again

Add this agentic workflow to your repo

To install this agentic workflow, run

gh aw add githubnext/agentics/workflows/repo-assist.md@b4e175bab6674279e0fe158f7dc877b77bc9d92f

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@fpellet
fpellet marked this pull request as ready for review October 17, 2026 10:09
@fpellet
fpellet merged commit 504f35b into master Oct 17, 2026
4 checks passed
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.

paket restore on slnx doesn't work

1 participant