Skip to content

Correct case for 'pi' in area calculation challenge#54471

Open
mwiemer-microsoft wants to merge 2 commits into
mainfrom
mwiemer-microsoft-patch-7
Open

Correct case for 'pi' in area calculation challenge#54471
mwiemer-microsoft wants to merge 2 commits into
mainfrom
mwiemer-microsoft-patch-7

Conversation

@mwiemer-microsoft

@mwiemer-microsoft mwiemer-microsoft commented Jun 17, 2026

Copy link
Copy Markdown
Member

Copilot AI review requested due to automatic review settings June 17, 2026 23:47
@mwiemer-microsoft mwiemer-microsoft requested review from a team and BillWagner as code owners June 17, 2026 23:47
@dotnetrepoman dotnetrepoman Bot added this to the June 2026 milestone Jun 17, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the “numbers in C#” tutorial to use the correct casing for the mathematical constant pi in the circle-area challenge, while still pointing learners to the System.Math.PI constant.

Changes:

  • Replaces “PI” with “pi” in the area-of-a-circle challenge text.
  • Updates the accompanying hint text to refer to “pi” consistently.

Comment thread docs/csharp/tour-of-csharp/tutorials/numbers-in-csharp.md Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

@BillWagner BillWagner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks @mwiemer-microsoft

This LGTM, and I'll :shipit: as soon as the build finishes.

@BillWagner BillWagner enabled auto-merge (squash) June 18, 2026 13:04
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.

3 participants