Skip to content

Template update#1105

Merged
alfonsogarciacaro merged 1 commit intomasterfrom
template_update
Aug 6, 2017
Merged

Template update#1105
alfonsogarciacaro merged 1 commit intomasterfrom
template_update

Conversation

@MangelMaxime
Copy link
Copy Markdown
Member

  • Update the deps to latest versions
  • Use paket to handle FSharp.NET.Sdk

I propose to manage FSharp.NET.Sdk via paket because there seems to have issues when opening a clean template into VS2017. And like that alls the deps are manage by paket or npm/yarn.

@alfonsogarciacaro I already updated the version in the .proj file

@alfonsogarciacaro
Copy link
Copy Markdown
Member

True, better to manage FSharp.NET.Sdk with Paket. But it's still not possible to open the new .fsproj format with VS 2017 yet, right?

@alfonsogarciacaro alfonsogarciacaro merged commit 119b557 into master Aug 6, 2017
@MangelMaxime
Copy link
Copy Markdown
Member Author

I believe it's opening now. I don't use VS anymore myself.

cc @Zaid-Ajaj

@Zaid-Ajaj
Copy link
Copy Markdown
Member

@alfonsogarciacaro Nope, VS2017 on Windows seems to be looking for a globally installed FSharp.NET.Sdk on the machine that you don't get even you install the "F# Language Support". Installing the SDK locally with the project dependencies, VS2017 still complains and can't load the project. VS2017 really not worth it, VS Code "just works"

@alfonsogarciacaro alfonsogarciacaro deleted the template_update branch November 30, 2017 20:36
github-actions bot added a commit that referenced this pull request Apr 2, 2026
…patterns

- Replace unsafe `.Value` on option types with safe pattern matching
  in Python/Replacements.fs (#1203) and Rust/Fable2Rust.fs (#1126, #1127)
- Add [<return: Struct>] to partial active patterns in Dart/Fable2Dart.fs
  (#1105, #1106), Dart/Replacements.fs (#1177), and Rust/Fable2Rust.fs
  (#1115, #1116, #1117, #1118, #1119, #1120, #1121, #1122, #1123, #1124)
  to reduce heap allocations

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants