Skip to content

Conversation

@smoothdeveloper
Copy link
Contributor

@smoothdeveloper smoothdeveloper commented Jul 18, 2023

Adding note for users of .NET Framework, on how to invoke FSI.

I support applications and APIs targeting .NET Framework and the documentation refer to this page. I'd like to have those instructions laid out on the page.


Internal previews

📄 File 🔗 Preview link
docs/fsharp/tools/fsharp-interactive/index.md Interactive programming with F#

adding note for users of .NET Framework, on how to invoke FSI.
@smoothdeveloper smoothdeveloper requested review from a team and KathleenDollard as code owners July 18, 2023 14:53
@dotnet-bot dotnet-bot added this to the July 2023 milestone Jul 18, 2023
@ghost ghost added the community-contribution Indicates PR is created by someone from the .NET community. label Jul 18, 2023
Copy link
Member

@BillWagner BillWagner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for making this addition @smoothdeveloper

I added one comment to use our "NOTE" styling, so this gets highlighted in the published version.

Once you accept that suggestion, this is ready to merge.

Thanks again.

@smoothdeveloper
Copy link
Contributor Author

@BillWagner, thanks for the adjustment!

I'm ok for this to be merged, but also want to make sure nothing contentious (in context of message around .NET Framework).

My motive is just to make it easier on end users, that I can only support under .NET Framework for the time being.

Copy link
Member

@BillWagner BillWagner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@BillWagner BillWagner merged commit 737cdb3 into dotnet:main Jul 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-contribution Indicates PR is created by someone from the .NET community. dotnet-fsharp/svc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants