This repository has been archived by the owner on Jan 3, 2023. It is now read-only.
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Showing
86 changed files
with
13,170 additions
and
694 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,6 @@ | ||
| <?xml version="1.0" encoding="utf-8"?> | ||
| <configuration> | ||
| <solution> | ||
| <add key="disableSourceControlIntegration" value="true" /> | ||
| </solution> | ||
| </configuration> |
Binary file not shown.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,136 @@ | ||
| <?xml version="1.0" encoding="utf-8"?> | ||
| <Project ToolsVersion="4.0" xmlns="http://schemas.microsoft.com/developer/msbuild/2003"> | ||
| <PropertyGroup> | ||
| <SolutionDir Condition="$(SolutionDir) == '' Or $(SolutionDir) == '*Undefined*'">$(MSBuildProjectDirectory)\..\</SolutionDir> | ||
|
|
||
| <!-- Enable the restore command to run before builds --> | ||
| <RestorePackages Condition=" '$(RestorePackages)' == '' ">false</RestorePackages> | ||
|
|
||
| <!-- Property that enables building a package from a project --> | ||
| <BuildPackage Condition=" '$(BuildPackage)' == '' ">false</BuildPackage> | ||
|
|
||
| <!-- Determines if package restore consent is required to restore packages --> | ||
| <RequireRestoreConsent Condition=" '$(RequireRestoreConsent)' != 'false' ">true</RequireRestoreConsent> | ||
|
|
||
| <!-- Download NuGet.exe if it does not already exist --> | ||
| <DownloadNuGetExe Condition=" '$(DownloadNuGetExe)' == '' ">false</DownloadNuGetExe> | ||
| </PropertyGroup> | ||
|
|
||
| <ItemGroup Condition=" '$(PackageSources)' == '' "> | ||
| <!-- Package sources used to restore packages. By default, registered sources under %APPDATA%\NuGet\NuGet.Config will be used --> | ||
| <!-- The official NuGet package source (https://www.nuget.org/api/v2/) will be excluded if package sources are specified and it does not appear in the list --> | ||
| <!-- | ||
| <PackageSource Include="https://www.nuget.org/api/v2/" /> | ||
| <PackageSource Include="https://my-nuget-source/nuget/" /> | ||
| --> | ||
| </ItemGroup> | ||
|
|
||
| <PropertyGroup Condition=" '$(OS)' == 'Windows_NT'"> | ||
| <!-- Windows specific commands --> | ||
| <NuGetToolsPath>$([System.IO.Path]::Combine($(SolutionDir), ".nuget"))</NuGetToolsPath> | ||
| <PackagesConfig>$([System.IO.Path]::Combine($(ProjectDir), "packages.config"))</PackagesConfig> | ||
| </PropertyGroup> | ||
|
|
||
| <PropertyGroup Condition=" '$(OS)' != 'Windows_NT'"> | ||
| <!-- We need to launch nuget.exe with the mono command if we're not on windows --> | ||
| <NuGetToolsPath>$(SolutionDir).nuget</NuGetToolsPath> | ||
| <PackagesConfig>packages.config</PackagesConfig> | ||
| </PropertyGroup> | ||
|
|
||
| <PropertyGroup> | ||
| <!-- NuGet command --> | ||
| <NuGetExePath Condition=" '$(NuGetExePath)' == '' ">$(NuGetToolsPath)\NuGet.exe</NuGetExePath> | ||
| <PackageSources Condition=" $(PackageSources) == '' ">@(PackageSource)</PackageSources> | ||
|
|
||
| <NuGetCommand Condition=" '$(OS)' == 'Windows_NT'">"$(NuGetExePath)"</NuGetCommand> | ||
| <NuGetCommand Condition=" '$(OS)' != 'Windows_NT' ">mono --runtime=v4.0.30319 $(NuGetExePath)</NuGetCommand> | ||
|
|
||
| <PackageOutputDir Condition="$(PackageOutputDir) == ''">$(TargetDir.Trim('\\'))</PackageOutputDir> | ||
|
|
||
| <RequireConsentSwitch Condition=" $(RequireRestoreConsent) == 'true' ">-RequireConsent</RequireConsentSwitch> | ||
| <NonInteractiveSwitch Condition=" '$(VisualStudioVersion)' != '' AND '$(OS)' == 'Windows_NT' ">-NonInteractive</NonInteractiveSwitch> | ||
|
|
||
| <PaddedSolutionDir Condition=" '$(OS)' == 'Windows_NT'">"$(SolutionDir) "</PaddedSolutionDir> | ||
| <PaddedSolutionDir Condition=" '$(OS)' != 'Windows_NT' ">"$(SolutionDir)"</PaddedSolutionDir> | ||
|
|
||
| <!-- Commands --> | ||
| <RestoreCommand>$(NuGetCommand) install "$(PackagesConfig)" -source "$(PackageSources)" $(NonInteractiveSwitch) $(RequireConsentSwitch) -solutionDir $(PaddedSolutionDir)</RestoreCommand> | ||
| <BuildCommand>$(NuGetCommand) pack "$(ProjectPath)" -Properties "Configuration=$(Configuration);Platform=$(Platform)" $(NonInteractiveSwitch) -OutputDirectory "$(PackageOutputDir)" -symbols</BuildCommand> | ||
|
|
||
| <!-- We need to ensure packages are restored prior to assembly resolve --> | ||
| <BuildDependsOn Condition="$(RestorePackages) == 'true'"> | ||
| RestorePackages; | ||
| $(BuildDependsOn); | ||
| </BuildDependsOn> | ||
|
|
||
| <!-- Make the build depend on restore packages --> | ||
| <BuildDependsOn Condition="$(BuildPackage) == 'true'"> | ||
| $(BuildDependsOn); | ||
| BuildPackage; | ||
| </BuildDependsOn> | ||
| </PropertyGroup> | ||
|
|
||
| <Target Name="CheckPrerequisites"> | ||
| <!-- Raise an error if we're unable to locate nuget.exe --> | ||
| <Error Condition="'$(DownloadNuGetExe)' != 'true' AND !Exists('$(NuGetExePath)')" Text="Unable to locate '$(NuGetExePath)'" /> | ||
| <!-- | ||
| Take advantage of MsBuild's build dependency tracking to make sure that we only ever download nuget.exe once. | ||
| This effectively acts as a lock that makes sure that the download operation will only happen once and all | ||
| parallel builds will have to wait for it to complete. | ||
| --> | ||
| <MsBuild Targets="_DownloadNuGet" Projects="$(MSBuildThisFileFullPath)" Properties="Configuration=NOT_IMPORTANT;DownloadNuGetExe=$(DownloadNuGetExe)" /> | ||
| </Target> | ||
|
|
||
| <Target Name="_DownloadNuGet"> | ||
| <DownloadNuGet OutputFilename="$(NuGetExePath)" Condition=" '$(DownloadNuGetExe)' == 'true' AND !Exists('$(NuGetExePath)')" /> | ||
| </Target> | ||
|
|
||
| <Target Name="RestorePackages" DependsOnTargets="CheckPrerequisites"> | ||
| <Exec Command="$(RestoreCommand)" | ||
| Condition="'$(OS)' != 'Windows_NT' And Exists('$(PackagesConfig)')" /> | ||
|
|
||
| <Exec Command="$(RestoreCommand)" | ||
| LogStandardErrorAsError="true" | ||
| Condition="'$(OS)' == 'Windows_NT' And Exists('$(PackagesConfig)')" /> | ||
| </Target> | ||
|
|
||
| <Target Name="BuildPackage" DependsOnTargets="CheckPrerequisites"> | ||
| <Exec Command="$(BuildCommand)" | ||
| Condition=" '$(OS)' != 'Windows_NT' " /> | ||
|
|
||
| <Exec Command="$(BuildCommand)" | ||
| LogStandardErrorAsError="true" | ||
| Condition=" '$(OS)' == 'Windows_NT' " /> | ||
| </Target> | ||
|
|
||
| <UsingTask TaskName="DownloadNuGet" TaskFactory="CodeTaskFactory" AssemblyFile="$(MSBuildToolsPath)\Microsoft.Build.Tasks.v4.0.dll"> | ||
| <ParameterGroup> | ||
| <OutputFilename ParameterType="System.String" Required="true" /> | ||
| </ParameterGroup> | ||
| <Task> | ||
| <Reference Include="System.Core" /> | ||
| <Using Namespace="System" /> | ||
| <Using Namespace="System.IO" /> | ||
| <Using Namespace="System.Net" /> | ||
| <Using Namespace="Microsoft.Build.Framework" /> | ||
| <Using Namespace="Microsoft.Build.Utilities" /> | ||
| <Code Type="Fragment" Language="cs"> | ||
| <![CDATA[ | ||
| try { | ||
| OutputFilename = Path.GetFullPath(OutputFilename); | ||
| Log.LogMessage("Downloading latest version of NuGet.exe..."); | ||
| WebClient webClient = new WebClient(); | ||
| webClient.DownloadFile("https://www.nuget.org/nuget.exe", OutputFilename); | ||
| return true; | ||
| } | ||
| catch (Exception ex) { | ||
| Log.LogErrorFromException(ex); | ||
| return false; | ||
| } | ||
| ]]> | ||
| </Code> | ||
| </Task> | ||
| </UsingTask> | ||
| </Project> |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,123 @@ | ||
| ##Contribution Guidelines | ||
|
|
||
| The Visual F# team is proud to be a contributor to F#, and urge you to join in too. F# users and the F# community are grateful for all contributions to F#. | ||
|
|
||
| Besides this overview, we recommend ["Becoming a contributor"](http://mrange.wordpress.com/2014/12/11/becoming-an-fsharp-contributor/), a community blog post by Mårten Rånge. | ||
|
|
||
| ###Getting Started | ||
|
|
||
| - Install required software | ||
| - Clone the repo | ||
| - `git clone https://github.com/microsoft/visualfsharp.git` | ||
| - How to build | ||
| - How to run tests | ||
|
|
||
| ###What to Contribute? | ||
|
|
||
| There are several important ways that you can contribute. We are especially grateful for early feedback on in-development features, bug reports with repro steps, bug fixes with regression test cases, cross-platform expertise and changes, documentation updates, feature tests, suggestions, comments, and ideas. | ||
|
|
||
| We initially solicit contributions for | ||
|
|
||
| - compiler optimizations | ||
| - compiler performance improvements | ||
| - code generation improvements | ||
| - bug fixes (see the [issues list](https://github.com/microsoft/visualfsharp/issues)) | ||
| - library improvements | ||
| - F# language and library features | ||
|
|
||
| New features are welcome, but be aware that Visual F# is a high-quality programming language with high-quality tools, and we wish to keep it that way. Before embarking on an extensive feature implementation, make a proposal in a GitHub issue or on the [F# Language UserVoice](https://fslang.uservoice.com/) so the community can review and comment on it. | ||
|
|
||
| ### Issues | ||
|
|
||
| When submitting issues, please use the following guidelines | ||
|
|
||
| - Suggestions for the F# Language and Core library should be added and reviewed at the [F# Language User Voice](https://fslang.uservoice.com/). | ||
|
|
||
| - Suggestions for the Visual F# Tools should be added and reviewed at the [Visual Studio F# Tools User Voice](https://visualstudio.uservoice.com/forums/121579-visual-studio/category/30935-languages-f-tools). | ||
|
|
||
| - New Bug Reports should always give accurate, clear steps for reproducing the bug, and all relevant details about versions, platform, etc. We suggest the following template: | ||
|
|
||
| Title: <a short, clear title> | ||
|
|
||
| Description: <a description of the problem> | ||
|
|
||
| Repro Steps: <step by step description> | ||
|
|
||
| Expected: <what is expected> | ||
|
|
||
| Actual: <what you really get> | ||
|
|
||
| Severity: a description on how bad it is and why - is it blocking? | ||
|
|
||
| Version: Language, compiler, library, platform version | ||
|
|
||
| Link: Give a link to a ZIP, log files or other files if needed | ||
|
|
||
| Likely Cause: Link to the place in the code where things are likely going wrong, if possible | ||
|
|
||
| Workaround: List any known workarounds | ||
|
|
||
| ###CLA | ||
|
|
||
| Contributors are required to sign a [Contribution License Agreement](https://cla.msopentech.com/) (CLA) before any pull requests will be considered. After submitting a request via the provided form, electronically sign the CLA when you receive the email containing the link to the document. This only needs to be done once for each Microsoft OSS project you contribute to. | ||
|
|
||
| ###Quality and Testing | ||
|
|
||
| Contributions to this repository will be rigorously policed for quality. | ||
|
|
||
| All code submissions should be submitted with regression test cases, and will be subject to peer review by the community and Microsoft. The bar for contributions will be high. This will result in a higher-quality, more stable product. | ||
|
|
||
| - We expect contributors to be actively involved in quality assurance. | ||
| - We expect contributors to be actively involved in quality assurance. | ||
| - We expect contributors to be actively involved in quality assurance. | ||
| - Partial, incomplete, or poorly-tested contributions will not be accepted. | ||
| - Contributions may be put on hold according to stability, testing, and design-coherence requirements. | ||
|
|
||
| #### Mimimum Bar for Code Cleanup Pull Requests | ||
|
|
||
| In addition to the above, "Code Cleanup" pull requests have the following minimum requirements: | ||
|
|
||
| - There must be no chance of a behavioural change, performance degradation or regression under any reasonable reading of the code in the context of the codebase as a whole. | ||
|
|
||
| - Code cleanup which is unrelated to a bug fix or feature should generally be made separate to other checkins where possible. | ||
| - Code cleanup is much more likely to be accepted towards the start of a release cycle. | ||
|
|
||
| #### Mimimum Bar for Performance Improvement Pull Requests | ||
|
|
||
| Performance improvement checkins have the following minimum requirements (in addition to the above) | ||
|
|
||
| - Performance tests and figures must be given, either in the PR or in the notes associated with the PR. PRs without performance figures will be closed with a polite request to please add them. | ||
|
|
||
| - The PR must show a reliable, substantive performance improvement that justifies the complexity introduced. For the compiler, performance improvements of ~1% are of interest. For the core library, it will depend on the routine in question. For the Visual F# tools, reactivity of the user interface will be of more interest than raw CPU performance. | ||
|
|
||
| #### Mimimum Bar for Bug Fix Pull Requests | ||
|
|
||
| Bug fix PRs have the following minimum requirements | ||
|
|
||
| - There must be a separate tracking bug entry in the public GitHub issues. A link should be given in the PR. PRs without a matching bug link will be closed with a polite request to please add it. | ||
|
|
||
| - The code changes must be reasonably minimal and as low-churn, non-intrusive as possible. Unrelated cleanup should be done in separate PRs (see above), and fixes should be as small as possible. Code cleanup that is part of making a clear and accurate fix is acceptable as part of a bug fix, but care should be taken that it doesn't obscure the fix itself. For example, renaming identifiers to be clearer in a way that would have avoided the original bug is acceptable, but care must still be taken that the actual fix is still apparent and reviewable in the overall diff for the fix. | ||
|
|
||
| - Thorough test cases must be included in the PR (unless tests already exist for a failing case). PRs without matching tests will be closed with a polite request to please add the tests. However, if you need help adding tests, please note this in the description of the change and people will guide you through where to add the tests. | ||
|
|
||
| #### Mimimum Bar for Feature Pull Requests | ||
|
|
||
| Feature PRs have the following minimum requirements: | ||
|
|
||
| - For F# Language and Library features, include a link to the [F# Language User Voice](http://fslang.uservoice.com) | ||
|
|
||
| - For Visual F# Tools features, include a link to the [Visual F# Tools User Voice](https://visualstudio.uservoice.com/forums/121579-visual-studio/category/30935-languages-f-tools) entry for the feature. | ||
|
|
||
| - For F# Library features, if you have made additions to the FSharp.Core library public surface area, update [the SurfaceArea tests](https://github.com/Microsoft/visualfsharp/tree/fsharp4/src/fsharp/FSharp.Core.Unittests). | ||
|
|
||
| - For F# Language and Library features, you will be asked to submit a speclet for the feature to the [F# Language Design](https://github.com/fsharp/FSharpLangDesign/) GitHub repository of speclets. In some cases you will only need to do this after your feature is accepted, but for more complex features you may be asked to do this during the review of the feature. | ||
|
|
||
| ###Language Evolution | ||
|
|
||
| We are committed to carefully managing the evolution of the F# language. | ||
|
|
||
| We actively solicit contributions related to the F# language design, but the process for handling these differs substantially from other kinds of contributions. Significant language and library change should be suggested and reviewed at the [F# Language User Voice](https://fslang.uservoice.com/) site. | ||
|
|
||
| ###Coding guidelines | ||
|
|
||
| Although there is currently no strict set of coding or style guidelines, use common sense when contributing code - make an effort to use a similar style to nearby existing code. If you have a passion for helping us develop a set of coding guidelines that we can roll out and apply within this project, get involved and start a discussion issue. |
Oops, something went wrong.