Skip to content

src/Compiler/TypedTree/TypedTreePickle.fs and refactored. - #18203

Merged
T-Gro merged 3 commits into
dotnet:mainfrom
progressive-galib:18061-TypedTreePickle
Jan 7, 2025
Merged

src/Compiler/TypedTree/TypedTreePickle.fs and refactored.#18203
T-Gro merged 3 commits into
dotnet:mainfrom
progressive-galib:18061-TypedTreePickle

Conversation

@progressive-galib

@progressive-galibprogressive-galib commented Jan 6, 2025

Copy link
Copy Markdown
Contributor

Description

"#if" directive around nullness removed from src/Compiler/TypedTree/TypedTreePickle.fs and refactored.

Related: #18061 (partially addresses)

Checklist

  • Release notes entry updated: in
    docs/release-notes/.FSharp.Compiler.Service/9.0.200.md,

"#if" directive around nullness removed from src/Compiler/TypedTree/TypedTreePickle.fs and refactored.
Fixesdotnet#18061 (partially)
@progressive-galib
progressive-galib requested a review from a team as a code ownerJanuary 6, 2025 12:16
@github-actions

github-actionsBot commented Jan 6, 2025

Copy link
Copy Markdown
Contributor

❗ Release notes required


✅ Found changes and release notes in following paths:

Change pathRelease notes pathDescription
src/Compilerdocs/release-notes/.FSharp.Compiler.Service/9.0.200.md

Comment threaddocs/release-notes/.FSharp.Compiler.Service/9.0.200.md

@psfinakipsfinaki left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks!

@psfinaki

Copy link
Copy Markdown
Contributor

Note, I am editing your PR descriptions in other to not close the related ticket accidentally. GitHub has some voodoo which automatically closes tickets if it detects a sentence like "Fixes #XYZ".

@progressive-galib

Copy link
Copy Markdown
ContributorAuthor

@T-Gro
T-Gro merged commit 73b8cbb into dotnet:mainJan 7, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants

@progressive-galib@psfinaki@T-Gro