Skip to content

Add XML comments to all public members#561

Merged
codebude merged 4 commits intomasterfrom
xml_comments
Jun 3, 2024
Merged

Add XML comments to all public members#561
codebude merged 4 commits intomasterfrom
xml_comments

Conversation

@Shane32
Copy link
Owner

@Shane32 Shane32 commented Jun 2, 2024

Summary

This PR adds XML comments to all public members

No code has been changed.

Test Plan

CS1591 is enforced so any public members added in the future will require XML comments or will break the build.

Notes

These xml comments may not be perfect, but they're better than nothing. Please suggest any changes desired.

Copy link
Collaborator

@codebude codebude left a comment

Choose a reason for hiding this comment

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

Wow, that looks like a lot of work. Thanks! Before merging I kindly ask you to fix the test cases, too.

@codebude codebude merged commit d7b9c81 into Shane32:master Jun 3, 2024
@Shane32 Shane32 deleted the xml_comments branch June 3, 2024 13:56
@Shane32 Shane32 added this to the 1.6.0 milestone Oct 8, 2025
@Shane32 Shane32 added the documentation Project documentation or source code documentation label Oct 8, 2025
vaernion pushed a commit to Arbeidstilsynet/brevgen2 that referenced this pull request Dec 3, 2025
This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [QRCoder](https://github.com/codebude/QRCoder) | `1.4.3` -> `1.6.0` | [![age](https://developer.mend.io/api/mc/badges/age/nuget/QRCoder/1.6.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/nuget/QRCoder/1.6.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/nuget/QRCoder/1.4.3/1.6.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/nuget/QRCoder/1.4.3/1.6.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

<details>
<summary>codebude/QRCoder (QRCoder)</summary>

### [`v1.6.0`](https://github.com/codebude/QRCoder/releases/tag/v1.6.0)

#### What has changed

-   \[Enhancement] Huge performance (runtime and memory) optimization in `QrCodeGenerator` (PRs [#&#8203;509](Shane32/QRCoder#509), [#&#8203;519](Shane32/QRCoder#519), [#&#8203;520](Shane32/QRCoder#520), [#&#8203;521](Shane32/QRCoder#521), [#&#8203;524](Shane32/QRCoder#524), [#&#8203;530](Shane32/QRCoder#530), [#&#8203;532](Shane32/QRCoder#532))
-   \[Enhancement] Big performance (runtime and memory) optimization in `BitmapByteQRCode` (PR [#&#8203;566](Shane32/QRCoder#566))
-   \[Enhancement] Added ECC level mismatch detection when using payloads from payload generator (PR [#&#8203;526](Shane32/QRCoder#526))
-   \[Enhancement] Implementation of encoding constraints (EccLevel, EciMode) in SwissQRCode and GiroCode payloads to enforce that the payloads comply with the specifications of the respective standards (PR [#&#8203;533](Shane32/QRCoder#533))
-   \[Enhancement] Added support for trimming (`IsTrimmable` project property) (PR [#&#8203;539](Shane32/QRCoder#539))
-   \[Enhancement] Added XML comments (code documentation) to all public members (PR [#&#8203;561](Shane32/QRCoder#561))
-   \[Enhancement] Added new/third overload to `PngByteQRCode` that accepts colors as System.Drawing.Color (PR [#&#8203;564](Shane32/QRCoder#564))
-   \[Enhancement] Updated the Bitmap header from BITMAPCOREHEADER to BITMAPINFOHEADER in `BitmapByteQRCode` (PR [#&#8203;565](Shane32/QRCoder#565))
-   \[Refactoring] Restructured `QRCodeGenerator` classes into separate files and partially restructured code for better manageability (PRs [#&#8203;516](Shane32/QRCoder#516), [#&#8203;528](Shane32/QRCoder#528), [#&#8203;531](https://github....
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Project documentation or source code documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants