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.

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

logf should support interpolated strings #7

Open
Tracked by #25
jwosty opened this issue Jun 8, 2023 · 2 comments · Fixed by #25
Open
Tracked by #25

logf should support interpolated strings #7

jwosty opened this issue Jun 8, 2023 · 2 comments · Fixed by #25
Labels
blocked enhancement New feature or request impact:MEDIUM

Comments

@jwosty
Copy link
Owner

jwosty commented Jun 8, 2023 •
edited
Loading

The following should work:

let durationMs = 5. / 3.
logfi ml $"Duration: %.2f{durationMs}ms"

Currently, this blows up with a runtime exception.

IIRC, this might actually be impossible to implement at the moment. Will investigate again to verify.
@jwosty jwosty added enhancement New feature or request impact:LOW impact:MEDIUM and removed impact:LOW labels Jun 8, 2023
@jwosty jwosty added the blocked label Jun 28, 2023
@jwosty
Copy link
Owner Author

jwosty commented Jun 28, 2023

Blocked by: vbfox/MasterOfFoo#5

@jwosty jwosty mentioned this issue Dec 4, 2024
Merged
4 tasks
@jwosty jwosty linked a pull request Dec 4, 2024 that will close this issue
.NET 9 #25
Merged
4 tasks
@jwosty
Copy link
Owner Author

jwosty commented Dec 4, 2024 •
edited
Loading

Working now. Fable implementation blocked by: fable-compiler/Fable#3969

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
blocked enhancement New feature or request impact:MEDIUM
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

.NET 9 jwosty/FSharp.Logf
1 participant
@jwosty