Skip to content

docs: correct SDK prerequisite to .NET 10.0+ in README #35

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.

Sign up for GitHub

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
rubenhensen merged 2 commits into from
Jun 23, 2026
Merged

docs: correct SDK prerequisite to .NET 10.0+ in README #35

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
cc0a8de
docs: correct SDK prerequisite to .NET 10.0+ in README
dobby-coder[bot] Jun 16, 2026
e0bf855
docs: drop justification parenthetical from SDK prerequisite
dobby-coder[bot] Jun 16, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ See the [full API reference](https://docs.postguard.eu/repos/postguard-dotnet) f

### Prerequisites

- .NET 8.0+ SDK
- .NET 10.0+ SDK (the projects multi-target `net8.0` and `net10.0`, so the SDK must support the highest target, `net10.0`)

@dobby-coder dobby-coder Bot Jun 16, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Rule check (no-justification-paragraphs-for-simple-changes): the parenthetical explaining why net10.0 is required is the kind of in-docs editorializing Ruben has asked us to drop for self-evident changes ("Just remove it and let that be that. No need to make a paragraph of text about it."). A reader of Prerequisites just needs the version, and issue #12 itself proposed the terse fix. Suggestion:

Suggested change
- .NET 10.0+ SDK (the projects multi-target `net8.0` and `net10.0`, so the SDK must support the highest target, `net10.0`)
- .NET 10.0+ SDK

The multi-target detail already lives in the .csproj and git history.

- Rust toolchain ([rustup](https://rustup.rs/))

### Build native library
Expand Down