Skip to content

Move operations from ExtraTopLevelOperators to a better place. #14847

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 4 commits into from
Closed

Move operations from ExtraTopLevelOperators to a better place. #14847

wants to merge 4 commits into from

Conversation

@KevinRansom
Copy link
Contributor

@KevinRansom KevinRansom commented Mar 4, 2023

  1. Refactoring to move the APIs to where they belong.
  2. Ensuring the compiler and Codegen use the moved version.
  3. Adding EditorNotBrowsable, so the original namespaces and functions are not proposed.
  4. Considering Obsoleting them

We want to try to organise FSharp.Core a little less monolithically, this is just the first step.

This PR is WIP: Just to see what is broken.

@KevinRansom KevinRansom requested a review from a team as a code owner March 4, 2023 03:14
@KevinRansom KevinRansom force-pushed the refactorpervasives branch 6 times, most recently from a717265 to 4e66fdf Compare March 6, 2023 10:31
@KevinRansom KevinRansom mentioned this pull request Mar 7, 2023
Merged
@KevinRansom KevinRansom force-pushed the refactorpervasives branch 7 times, most recently from 198d829 to 9b1e5fd Compare March 9, 2023 07:30
@KevinRansom KevinRansom closed this Mar 9, 2023
@KevinRansom KevinRansom deleted the refactorpervasives branch April 29, 2023 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

F# Compiler and Tooling
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@KevinRansom