-
Notifications
You must be signed in to change notification settings - Fork 870
Update several links #7055
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update several links #7055
Changes from 6 commits
b7719ca
950a870
0dba02e
c249819
bf5a6b3
cc1fd86
b734345
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,5 +1,3 @@ | ||
|
|
||
|
|
||
| # The FSharp.Compiler.Service components and NuGet package | ||
|
|
||
| This directory contains the build, packaging, test and documentation-generation logic for the NuGet package ``FSharp.Compiler.Service``. The source for this NuGet | ||
|
|
@@ -11,7 +9,7 @@ Basically we are packaging up the compiler as a DLL and publishing it as a NuGet | |
|
|
||
| There are subtle differences between FSharp.Compiler.Service and FSharp.Compiler.Private (shipped with the Visual F# Tools) | ||
|
|
||
| - FCS has a public API | ||
| - FCS has a public API | ||
|
|
||
| - FCS is built against **.NET 4.6.1** and **FSharp.Core NuGet 4.6.2** to give broader reach | ||
|
|
||
|
|
@@ -64,10 +62,9 @@ which does things like: | |
| You can push the packages if you have permissions, either automatically using ``build Release`` or manually | ||
|
|
||
| set APIKEY=... | ||
| ..\fsharp\.nuget\nuget.exe push %HOMEDRIVE%%HOMEPATH%\Downloads\FSharp.Compiler.Service.22.0.3.nupkg %APIKEY% -Source https://nuget.org | ||
| ..\fsharp\.nuget\nuget.exe push %HOMEDRIVE%%HOMEPATH%\Downloads\FSharp.Compiler.Service.22.0.3.nupkg %APIKEY% -Source https://nuget.org | ||
| ..\fsharp\.nuget\nuget.exe push %HOMEDRIVE%%HOMEPATH%\Downloads\FSharp.Compiler.Service.MSBuild.v12.22.0.3.nupkg %APIKEY% -Source https://nuget.org | ||
| ..\fsharp\.nuget\nuget.exe push %HOMEDRIVE%%HOMEPATH%\Downloads\FSharp.Compiler.Service.ProjectCracker.22.0.3.nupkg %APIKEY% -Source https://nuget.org | ||
|
|
||
|
|
||
| ### Use of Paket and FAKE | ||
|
|
||
|
|
@@ -77,23 +74,21 @@ FAKE is only used to run build.fsx. Eventually we will likely remove this once | |
|
|
||
| ### Testing | ||
|
|
||
| Testing reuses the test files from ..\tests\service which were are also FCS tests. | ||
|
|
||
| Testing reuses the test files from ..\tests\service which were are also FCS tests. | ||
|
|
||
| ### Documentation Generation | ||
|
|
||
| fcs\build GenerateDocs | ||
|
|
||
| Output is in ``docs``. In the ``FSharp.Compiler.Service`` repo this is checked in and hosted as http://fsharp.github.io/FSharp.Compiler.Service. | ||
|
|
||
| Output is in ``docs``. In the ``FSharp.Compiler.Service`` repo this is checked in and hosted as <https://fsharp.github.io/FSharp.Compiler.Service>. | ||
|
|
||
| ## The two other NuGet packages | ||
|
|
||
| It also contains both the source, build, packaging and test logic for | ||
| It also contains both the source, build, packaging and test logic for | ||
|
|
||
| * ``FSharp.Compiler.Service.MSBuild.v12`` adds legacy MSBuild v12 support to an instance of FSharp.Compiler.Service, if exact compatibility for scripting references such as ``#r "Foo, Version=1.3.4"`` is required. | ||
| - ``FSharp.Compiler.Service.MSBuild.v12`` adds legacy MSBuild v12 support to an instance of FSharp.Compiler.Service, if exact compatibility for scripting references such as ``#r "Foo, Version=1.3.4"`` is required. | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why this change is needed?
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. it was a complaint of mdlint in Visual Studio Code, where I accepted the proposed change to get rid of it. https://github.com/DavidAnson/markdownlint/blob/v0.15.0/doc/Rules.md#md004 |
||
|
|
||
| * ``FSharp.Compiler.Service.ProjectCracker`` is part of ``FsAutoComplete`` and Ionide and is used to crack old-style project formats using MSBuild. It used to be part of the FCS API. | ||
| - ``FSharp.Compiler.Service.ProjectCracker`` is part of ``FsAutoComplete`` and Ionide and is used to crack old-style project formats using MSBuild. It used to be part of the FCS API. | ||
|
|
||
| Both of these components are gradually becoming obsolete | ||
|
|
||
|
|
@@ -102,8 +97,7 @@ Both of these components are gradually becoming obsolete | |
| FSharp.Compiler.Service is a somewhat awkward component. There are some things we can do to simplify things: | ||
|
|
||
| 1. Remove the use of Paket and FAKE | ||
| 1. Move all projects under fcs\... to new .NET SDK project file format | ||
| 1. Move all projects under fcs\... to new .NET SDK project file format | ||
| 1. Drop the use of ``dotnet mergenupkg`` since we should be able to use cross targeting | ||
| 1. Make FCS a DLL similar ot the rest of the build and make this an official component from Microsoft (signed etc.) | ||
| 1. Replace FSharp.Compiler.Private by FSharp.Compiler.Service | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like a typo.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, thanks