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.

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

Add the type Fellows #84

Closed
Bullrich opened this issue Sep 19, 2023 · 0 comments · Fixed by #85
Closed

Add the type Fellows #84

Bullrich opened this issue Sep 19, 2023 · 0 comments · Fixed by #85
Assignees

Comments

@Bullrich
Copy link
Collaborator

Add a type for rules named fellows and move the rank system to that type so we can have specific rules for that situation and enhance it as needed.

Originally requested by @bkchr in polkadot-fellows/runtimes#31 (review)

@Bullrich Bullrich self-assigned this Sep 19, 2023
Bullrich added a commit that referenced this issue Sep 21, 2023
Created a new type of rule named `fellows`. 

This rule has its own evaluation (currently, it is mostly a copy of the
`basic` evaluation). This is done so we can add special cases for rule,
as _suggested by @bkchr in
polkadot-fellows/runtimes#31 (review).

Removed all the ability to request a `rank` in normal rules. I believe
that this is better as we now separate the concerns and simplify logic.
The `or`, `and` and `and-distinct` rule didn't really apply on fellows
because a higher ranking one belongs also to the lower echelons.

This resolves #84 and allows us to experiment better.

Also, this is technically a breaking change, but it has never been
applied on any system, so it wouldn't be breaking an existing system.
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 a pull request may close this issue.

1 participant