Skip to content

Fix csharp string interpolation in Timestamp.ToDateTime() exception message#26341

Closed
tyulpan wants to merge 2 commits into
protocolbuffers:mainfrom
tyulpan:fix-csharp-timestamp-exception-message
Closed

Fix csharp string interpolation in Timestamp.ToDateTime() exception message#26341
tyulpan wants to merge 2 commits into
protocolbuffers:mainfrom
tyulpan:fix-csharp-timestamp-exception-message

Conversation

@tyulpan

@tyulpan tyulpan commented Mar 12, 2026

Copy link
Copy Markdown
Contributor

Change a verbatim string literal @"" to an interpolated string $"" in Timestamp.ToDateTime() exception message.
Resolves #26006.

@jskeet

jskeet commented Mar 12, 2026

Copy link
Copy Markdown
Contributor

We already had a PR for this, #26270. I'm not sure what's happened to that at the moment. @mkruskal-google you approved that 4 days ago - do you know why it's not found any more?

@zhangskz

Copy link
Copy Markdown
Member

It looks like that PR failed a test due to a flake, and the user deleted their fork prior before the PR got unstuck / merged which closed the PR.

@zhangskz zhangskz added the 🅰️ safe for tests Mark a commit as safe to run presubmits over label Mar 17, 2026
@github-actions github-actions Bot removed the 🅰️ safe for tests Mark a commit as safe to run presubmits over label Mar 17, 2026
@jskeet
jskeet requested a review from mkruskal-google March 18, 2026 08:00
@karenwuz karenwuz added the 🅰️ safe for tests Mark a commit as safe to run presubmits over label Apr 7, 2026
@github-actions github-actions Bot removed the 🅰️ safe for tests Mark a commit as safe to run presubmits over label Apr 7, 2026
@esrauchg esrauchg added the 🅰️ safe for tests Mark a commit as safe to run presubmits over label Apr 21, 2026
@github-actions github-actions Bot removed the 🅰️ safe for tests Mark a commit as safe to run presubmits over label Apr 21, 2026
@tyulpan
tyulpan deleted the fix-csharp-timestamp-exception-message branch April 22, 2026 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Timestamp.ToDateTime() is not formatting the error message

5 participants