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

Make trimmed printf "%A" for F# records work when trimmed #14941

Merged
merged 5 commits into from
Apr 6, 2023

Conversation

KevinRansom
Copy link
Member

@KevinRansom KevinRansom commented Mar 20, 2023

  • Implement failing tests
  • Embedding DynamicallyAccessedMemberTypes
  • Adding DynamicallyAccessedMemberTypes with code generation to record type

@KevinRansom KevinRansom requested a review from a team as a code owner March 20, 2023 21:08
@KevinRansom KevinRansom changed the title Make trimmed pritnf "%A" for F# records work when trimmed Make trimmed printf "%A" for F# records work when trimmed Mar 20, 2023
@KevinRansom KevinRansom force-pushed the percenta.records branch 6 times, most recently from 7abaa84 to e1591f0 Compare March 27, 2023 02:27
@KevinRansom KevinRansom force-pushed the percenta.records branch 4 times, most recently from cdf8678 to ca587ea Compare March 31, 2023 16:43
@psfinaki psfinaki merged commit b1619b6 into dotnet:main Apr 6, 2023
vzarytovskii pushed a commit that referenced this pull request Apr 6, 2023
* Make trimmed printf "%A" for F# records work when trimmed (#14941)

* Add tests

* Embed types

* baselines + fsx

* Extra telemetry for codefixes (#15035)

* BDN config option - ignore that Test.Utilities is not optimized (#15038)

* Unignore files for formatting (.fantomasignore) if the underlying Fantomas issue has been resolved (#14960)

* SI.fs fantomas'd

* unignore .fsi files in Fsharp.Core

* Removing commented out files from fantomasignore

* Automated command ran: fantomas

  Co-authored-by: T-Gro <46543583+T-Gro@users.noreply.github.com>

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

---------

Co-authored-by: Kevin Ransom (msft) <codecutter@hotmail.com>
Co-authored-by: Petr <psfinaki@users.noreply.github.com>
Co-authored-by: Tomas Grosup <tomasgrosup@microsoft.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@KevinRansom KevinRansom deleted the percenta.records branch April 29, 2023 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants