Skip to content
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

Remove the WithPriority() deprecated method #133

Open
luispfgarces opened this issue Mar 15, 2023 · 0 comments
Open

Remove the WithPriority() deprecated method #133

luispfgarces opened this issue Mar 15, 2023 · 0 comments
Labels
low priority Low priority issue maintenance & refactoring Restructure the codebase without altering functionality

Comments

@luispfgarces
Copy link
Contributor

Is your request related to a problem you have?

Remove the WithPriority(...) method which has been deprecated for quite some time now, and we think it is safe to remove it.

image

Describe the solution you'd like

Removal of the method. Its functionality has been replaced by the RuleAddPriorityOption on RulesEngine.AddRuleAsync(...).

image

Are you able to help bring it to life and contribute with a Pull Request?

Yes

Additional context

No response
@luispfgarces luispfgarces added enhancement New feature or request maintenance & refactoring Restructure the codebase without altering functionality and removed enhancement New feature or request labels Mar 15, 2023
@Daniel-C-Dias Daniel-C-Dias added the low priority Low priority issue label May 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
low priority Low priority issue maintenance & refactoring Restructure the codebase without altering functionality
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Daniel-C-Dias @luispfgarces