Skip to content

Sync from Azure DevOps: release/dev17.12 #18741

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

Open
wants to merge 14 commits into
base: release/dev17.12
Choose a base branch
from

Conversation

T-Gro
Copy link
Member

@T-Gro T-Gro commented Jul 9, 2025

This PR syncs commits from the internal Azure DevOps mirror for branch 'release/dev17.12'. Please review and merge.

KevinRansom and others added 13 commits March 25, 2025 20:12
Update nuget handling in fsi
release/dev17.12

----
#### AI description  (iteration 1)
#### PR Classification
New feature and enhancement.

#### PR Summary
This pull request updates the handling of NuGet packages in F# Interactive (FSI) and introduces new event handling for text views.
- `Common/Extensions.fs`: Added `TextViewEventsHandler` and methods to handle text view events and caret positions.
- `LanguageService/FSharpProjectOptionsManager.fs`: Enhanced project options management with new event subscriptions and caret position handling.
- `Driver/ScriptClosure.fs`: Updated script closure processing to consider caret positions and package manager lines.
- `FSharp.DependencyManager.Nuget/FSharp.DependencyManager.fs`: Added support for `usepackagetargets` option in NuGet package references.
- `FSharp.Compiler.Private.Scripting.UnitTests/FSharpScriptTests.fs`: Added tests for the new `usepackagetargets` option in script evaluation.
<!-- GitOpsUserAgent=GitOps.Apps.Server.pullrequestcopilot -->
de-loc

----
#### AI description  (iteration 1)
#### PR Classification
Code cleanup

#### PR Summary
This pull request cleans up error handling and localization files by removing redundant translations and updating error messages.
- `FSharp.DependencyManager.fs`: Replaced specific boolean error message with a more generic `ArgumentOutOfRangeException` message.
- `FSDependencyManager.txt`: Removed the `invalidBooleanValue` error message.
- `FSharpScriptTests.fs`: Updated test cases to reflect the new error message format.
- Multiple localization files (`.xlf`): Removed the `invalidBooleanValue` translation entries.
<!-- GitOpsUserAgent=GitOps.Apps.Server.pullrequestcopilot -->
#### AI description  (iteration 1)
#### PR Classification
NuGet patch update and build configuration adjustments.

#### PR Summary
This pull request updates the NuGet package versions and adjusts build configurations to support internal source builds and dependencies. It ensures compatibility with the latest Microsoft Build version and adds new package sources.
- `eng/build.sh`: Added no-operation implementations for runtime arguments used in internal builds.
- `eng/SourceBuildPrebuiltBaseline.xml`: Added new usage entries for several Microsoft Build components and System.Formats.Asn1.
- `azure-pipelines.yml`: Enabled internal sources for source builds.
- `eng/Versions.props`: Updated `MicrosoftBuildVersion` to 17.12.36.
- `NuGet.config`: Added a new package source for `msbuild-Trusted`.
<!-- GitOpsUserAgent=GitOps.Apps.Server.pullrequestcopilot -->
Copy link
Contributor

github-actions bot commented Jul 9, 2025

⚠️ Release notes required, but author opted out

Warning

Author opted out of release notes, check is disabled for this pull request.
cc @dotnet/fsharp-team-msft

@github-project-automation github-project-automation bot moved this from New to In Progress in F# Compiler and Tooling Jul 9, 2025
@T-Gro T-Gro added the NO_RELEASE_NOTES Label for pull requests which signals, that user opted-out of providing release notes label Jul 9, 2025
@T-Gro
Copy link
Member Author

T-Gro commented Jul 11, 2025

/azp run

Copy link

Azure Pipelines successfully started running 2 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NO_RELEASE_NOTES Label for pull requests which signals, that user opted-out of providing release notes
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

3 participants