Skip to content

Add missing policies to PolicyDefinition #281

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

Jump to bottom
Merged
zidad merged 33 commits into from
Dec 11, 2023
Merged

Add missing policies to PolicyDefinition #281

zidad merged 33 commits into from
Dec 11, 2023

Conversation

@inikulshin
Copy link
Contributor

@inikulshin inikulshin commented Dec 10, 2023

Based on
image

zidad
zidad previously approved these changes Dec 11, 2023
View reviewed changes
@zidad
Copy link
Member

zidad commented Dec 11, 2023

Wow, many thanks!

@zidad zidad self-assigned this Dec 11, 2023
@inikulshin
Copy link
Contributor Author

inikulshin commented Dec 11, 2023 •
edited
Loading

Wow, many thanks!

However, the PR breaks public API.
I'll make JsonStringEnumConverterEx internal, but all other changes are mandatory.

Upd: @zidad do you want me to fix EasyNetQ.Management.Client.approved.txt?

@inikulshin
Copy link
Contributor Author

inikulshin commented Dec 11, 2023

Wow, many thanks!

@zidad sorry for many aftercommits (I hope you'll squash all them in merge). Can you recheck and review it?

@inikulshin
Copy link
Contributor Author

inikulshin commented Dec 11, 2023

Now it fails because of the last EOL symbol in EasyNetQ.Management.Client.approved.txt.
But I can't fix it from the browser, like all previous commits I did 😠

@zidad zidad self-requested a review December 11, 2023 19:56
@inikulshin
Copy link
Contributor Author

inikulshin commented Dec 11, 2023

Hallelujah!

I'll make some more PRs (at least 1) soon.
Will it be possible to create a new tag and publish new NuGet package soon?

Pliner
Pliner approved these changes Dec 11, 2023
View reviewed changes
Copy link
Member

@Pliner Pliner left a comment

Choose a reason for hiding this comment

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

Thanks!

@Pliner
Copy link
Member

Pliner commented Dec 11, 2023

Hallelujah!

I'll make some more PRs (at least 1) soon. Will it be possible to create a new tag and publish new NuGet package soon?

Sure. We will be able to release a new dev version available on NuGet.

@zidad zidad merged commit 7ca648c into EasyNetQ:master Dec 11, 2023
@Pliner
Copy link
Member

Pliner commented Dec 11, 2023

I have push a new tag, but GA is failed https://github.com/EasyNetQ/EasyNetQ.Management.Client/actions/runs/7173143825/job/19531930696.

Need to generate a new key...

@Pliner
Copy link
Member

Pliner commented Dec 11, 2023

https://www.nuget.org/packages/EasyNetQ.Management.Client/3.0.0-alpha3

@inikulshin inikulshin deleted the policydef branch December 13, 2023 16:11
sungam3r
sungam3r reviewed Dec 30, 2024
View reviewed changes
Source/EasyNetQ.Management.Client/Serialization/JsonStringEnumConverterEx.cs
// https://github.com/dotnet/runtime/issues/31081#issuecomment-848697673
internal class JsonStringEnumConverterEx<TEnum> : JsonConverter<TEnum> where TEnum : struct, Enum
{
private readonly Dictionary<TEnum, string> _enumToString = new Dictionary<TEnum, string>();
Copy link
Contributor

@sungam3r sungam3r Dec 30, 2024

Choose a reason for hiding this comment

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

All fields and constructor can be made static.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@zidad zidad zidad approved these changes

@Pliner Pliner Pliner approved these changes

@micdenny micdenny Awaiting requested review from micdenny micdenny is a code owner

@luigiberrettini luigiberrettini Awaiting requested review from luigiberrettini luigiberrettini is a code owner

+1 more reviewer

@sungam3r sungam3r sungam3r left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@zidad zidad

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@inikulshin @zidad @Pliner @sungam3r