Skip to content
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

Merge nightly into main for the release! #1180

Merged
merged 67 commits into from
Nov 16, 2023
Merged

Merge nightly into main for the release! #1180

merged 67 commits into from
Nov 16, 2023

Conversation

baronfel
Copy link
Contributor

@baronfel baronfel commented Oct 21, 2023

WHAT

🤖 Generated by Copilot at 63fa1ad

This pull request updates the FsAutoComplete project to support F# 6.0 features and use the latest FCS API. It modifies several files that deal with syntax analysis, code generation, code fixes, and testing to handle the new syntax, error codes, and record types introduced in F# 6.0. It also updates the dependencies to use compatible versions of FCS and its nightly feed.

🤖 Generated by Copilot at 63fa1ad

We're sailing on the F# 6.0 tide
We've got to update the FCS API
We'll match the new record syntax with pride
And fix the error codes on the count of three

🆕🛠️🗑️

WHY

We're close to release and there's a pretty stable set of packages. Lets go ahead and sync to main now that we're at that point in the lifecycle.

HOW

🤖 Generated by Copilot at 63fa1ad

  • Update FCS dependency and source URL to support F# 6.0 features (link, link)
  • Adapt to new record syntax for various syntax types in FCS API (link, link, link, link, link, link, link, link, link, link, link, link, link)
  • Handle new syntax cases for dot lambda expressions, intersection types, and while bang expressions in AST walking and error checking functions (src/FsAutoComplete.Core/UntypedAstUtils.fs, src/FsAutoComplete.Core/FCSPatches.fs) (link, link, link, link, link)
  • Pass source file kind argument to FSharpSourceTokenizer as required by new FCS API (src/FsAutoComplete.Core/Lexer.fs) (link)
  • Use setter symbol instead of getter symbol for property signature data (src/FsAutoComplete.Core/ParseAndCheckResults.fs) (link)
  • Assign priority to suggested name completion items (src/FsAutoComplete.Core/ParseAndCheckResults.fs) (link)
  • Remove cases for deprecated char range patterns in union case generation and matching functions (src/FsAutoComplete.Core/UnionPatternMatchCaseGenerator.fs) (link, link)
  • Remove local variables for checking extension and alias types, as FCS API handles this internally (src/FsAutoComplete.Core/UntypedAstUtils.fs) (link, link, link, link)
  • Add unit value to namespace declaration case in AST walking function (src/FsAutoComplete.Core/UntypedAstUtils.fs) (link)
  • Add empty line to the end of file (src/FsAutoComplete.Core/UntypedAstUtils.fs) (link)
  • Update diagnostic codes for missing equals in type definitions and missing dependency manager, as part of F# 6.0 error code renumbering (src/FsAutoComplete/CodeFixes/AddMissingEqualsToTypeDefinition.fs, test/FsAutoComplete.Tests.Lsp/CodeFixTests/Tests.fs, test/FsAutoComplete.Tests.Lsp/ScriptTests.fs) (link, link, link, link)

@baronfel baronfel enabled auto-merge (squash) October 21, 2023 19:14
@baronfel baronfel disabled auto-merge October 21, 2023 19:14
@baronfel baronfel enabled auto-merge October 21, 2023 19:15
* Update FCS to 43.8.100-preview.23523.6

* Pass in proper constant.

* Update to 43.8.100 stable
auto-merge was automatically disabled November 16, 2023 13:00

Pull Request is not mergeable

@baronfel
Copy link
Contributor Author

@nojaf you beautiful, wonderful, AMAZING human! I was struggling for so long trying to find that root cause.

Let's merge this thing!

@baronfel
Copy link
Contributor Author

Have to make a merge commit because we're merging the nightly branch in and I don't want future nightly -> main merges to lose history.

@baronfel baronfel merged commit a3f4ed3 into main Nov 16, 2023
20 of 25 checks passed
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.

5 participants