-
Notifications
You must be signed in to change notification settings - Fork 1.6k
System.DateTime F# snippets #7548
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
System.DateTime F# snippets #7548
Conversation
Docs Build status updates of commit 95da4a8: ✅ Validation status: passed
This comment lists only the first 50 files in the pull request. Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report. For any questions, please:
|
Adding @dsyme for review. I reviewed much of the code, and everything looks good to me. I don't think it needs a close review, but I'd appreciate a look at some samples and checking for recommended F# idioms. |
Docs Build status updates of commit 6f30d23: ✅ Validation status: passed
This comment lists only the first 50 files in the pull request. Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report. For any questions, please:
|
Docs Build status updates of commit 5c60da1: ✅ Validation status: passed
This comment lists only the first 50 files in the pull request. Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report. For any questions, please:
|
Docs Build status updates of commit 092409a: ✅ Validation status: passed
This comment lists only the first 50 files in the pull request. Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report. For any questions, please:
|
Amazing work!!! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM as well. Again, thanks @albert-du
Adds F# snippets for DateTime