Skip to content

prepare CHANGELOG.md for the next release#578

Merged
marcin-krystianc merged 3 commits intoG-Research:masterfrom
MohamedM216:next-release-1
Feb 24, 2026
Merged

prepare CHANGELOG.md for the next release#578
marcin-krystianc merged 3 commits intoG-Research:masterfrom
MohamedM216:next-release-1

Conversation

@MohamedM216
Copy link
Contributor

Description

Updating changelog for the next release.

Related Issues

Additional Context

Checklist

Please make sure to check the following before submitting your pull request:

  • Did you read the contributing guidelines?
  • Did you update the docs?
  • Did you write any tests to validate this change?

Copy link
Contributor

@marcin-krystianc marcin-krystianc left a comment

Choose a reason for hiding this comment

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

Feel free to mention adding public API analyzers, and also changing the public API to deal with RS0022 warnings.
In principle, these are not user-facing changes, but still may affect users if we've made unintentional mistakes, so it is worth letting them know.

CHANGELOG.md Outdated
Comment on lines 8 to 11
* Add implement ACL Binding Rule "Read" Endpoint (#550)
* Add implement ACL Binding Rule Update Endpoint (#551)
* Add implement ACL Binding Rule Delete Endpoint (#552)
* Add implement ACL Binding Rule List Endpoint (#556)
Copy link
Contributor

Choose a reason for hiding this comment

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

I think that the single "Implement" in place of "Add implement" would be better.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

👍

@MohamedM216
Copy link
Contributor Author

and also changing the public API to deal with RS0022 warnings.

RS0022 are still not resolved, should I still add it to CHANGELOG.md?

@marcin-krystianc
Copy link
Contributor

and also changing the public API to deal with RS0022 warnings.

RS0022 are still not resolved, should I still add it to CHANGELOG.md?

🤦 I meant RS0026

CHANGELOG.md Outdated
Comment on lines 14 to 28
* Fix RS0026 patch 1 (#559)
* Fix RS0026 errors patch 2 (#560)
* Fix RS0026 patch 3 (#561)
* Fix RS0026 patch 4 (#562)
* Fix RS0026 patch 5 (#563)
* Fix RS0026 patch 6 (#564)
* Fix RS0026 patch 7 (#566)
* Fix RS0026 patch 8 (#567)
* Fix RS0026 patch 9 (#568)
* Fix RS0026 patch 10 (#569)
* Fix RS0026 patch 11 (#570)
* Fix RS0026 patch 12 (#571)
* Fix RS0026 patch 14 (#573)
* Fix RS0026 final patch (#575)
* Fix RS0026 patch 13 (#574)
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's collapse it into single line and link all PRs (#559), (#560), etc...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK

CHANGELOG.md Outdated
Comment on lines 12 to 13
* Add Microsoft.CodeAnalysis.PublicApiAnalyzers (disabled warnings) (#554)
* Fix RS0016 errors and enable correct PublicAPI file generation (#557)
Copy link
Contributor

Choose a reason for hiding this comment

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

I would collapse it into single line too (just adding the Microsoft.CodeAnalysis.PublicApiAnalyzers

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK

@marcin-krystianc marcin-krystianc merged commit bbb2607 into G-Research:master Feb 24, 2026
231 of 234 checks passed
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