Skip to content

Update StringToParse section #7322

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

Merged
merged 3 commits into from
Mar 24, 2025
Merged

Update StringToParse section #7322

merged 3 commits into from
Mar 24, 2025

Conversation

timheuer
Copy link
Member

The information relating to the DateTimeFormatInfo object is important enough to call out earlier here in the docs...some of this information is later referenced only when referring to ParseExact, but this small addition is helpful to immediately note that the full DateTimeFormatInfo object is used as the default parsing, which would include formatting patterns.

The information relating to the `DateTimeFormatInfo` object is important enough to call out earlier here in the docs...some of this information is later referenced only when referring to ParseExact, but this small addition is helpful to immediately note that the full DateTimeFormatInfo object is used as the default parsing, which would include formatting patterns.
@timheuer timheuer requested a review from a team as a code owner October 26, 2021 02:30
@ghost ghost added the area-System.Runtime label Oct 26, 2021
@opbld31
Copy link

opbld31 commented Oct 26, 2021

Docs Build status updates of commit 1def2ee:

✅ Validation status: passed

File Status Preview URL Details
xml/System/DateTime.xml ✅Succeeded View

For more details, please refer to the build report.

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:

Copy link
Contributor

@gewarren gewarren left a comment

Choose a reason for hiding this comment

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

LGTM

@eerhardt
Copy link
Member

fyi - @tarekgh

Single-spacing after period

Co-authored-by: Eric Erhardt <[email protected]>
@opbld30
Copy link

opbld30 commented Oct 28, 2021

Docs Build status updates of commit f6061ff:

✅ Validation status: passed

File Status Preview URL Details
xml/System/DateTime.xml ✅Succeeded View

For more details, please refer to the build report.

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:

@tarekgh
Copy link
Member

tarekgh commented Mar 24, 2025

@timheuer @gewarren this is very old PR. I have resolved the conflict with the main and updated the text a little. Please let me know if you are ok with this or want to change anything. Thanks!

Copy link

Learn Build status updates of commit b5103ed:

✅ Validation status: passed

File Status Preview URL Details
xml/System/DateTime.xml ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

@tarekgh tarekgh added this to the Backlog milestone Mar 24, 2025
@tarekgh tarekgh enabled auto-merge March 24, 2025 23:08
@tarekgh tarekgh merged commit 52031fa into main Mar 24, 2025
4 checks passed
@tarekgh tarekgh deleted the timheuer-dtparsepatch branch March 24, 2025 23:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants