Skip to content

Dokumentations- und Codebereinigung - #7

Merged
Hanibal1963 merged 2 commits into
mainfrom
Entwurf
Oct 12, 2026
Merged

Hanibal1963 merged 2 commits into
mainfrom
Entwurf

Conversation

@Hanibal1963

Copy link
Copy Markdown
Owner

PR Classification

Dokumentations- und Codebereinigung: Das README-Beispiel wurde korrigiert, damit Sphere.Volume(...) direkt im Beispiel verwendbar ist.

PR Summary

Das PR bereinigt das README-Beispiel für die 3D-Geometrie und ergänzt den fehlenden Import für den direkten Zugriff auf Sphere.Volume(...). Zusätzlich wurden doppelte bzw. fehlerhafte Zeilen im Codeblock entfernt, um das Beispiel konsistent und lauffähig zu machen.

  • README.md: Import SchlumpfSoft.GeometryNET.ThreeDimensional ergänzt und fehlerhafte/doppelte Codezeilen im Beispiel bereinigt.

SchlumpfSoft.GeometryNET.ThreeDimensional in der README-Beispielanwendung importiert und doppelte Beispielzeilen für Berechnungen sowie Console.WriteLine-Ausgaben entfernt.
Projektdatei-Paket-Tag auf Trapezoid korrigiert und README-Kompatibilitätsangaben präzisiert (.NET Framework 4.8.1, .NET 8/9/10 Windows-spezifisch).
Copilot AI balanced review requested due to automatic review settings October 12, 2026 18:02
@Hanibal1963
Hanibal1963 merged commit 09aeabc into main Oct 12, 2026
1 check passed

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.

🟢 Approval recommended

The documentation example is consistent and references valid APIs.

0 open findings

What changed in this PR

Cleans the VB.NET quick-start example for consistent, directly usable geometry calls.

Changes:

  • Removes duplicate console output statements.
  • Normalizes the rectangle and sphere declarations.
File Description
README.md Cleans duplicate lines in the usage example.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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.

2 participants