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
Typing annotations: rule.py
#6516
Conversation
8cd2358
to
511512c
Compare
511512c
to
9ee5870
Compare
9ee5870
to
b41d555
Compare
I am squashing the commits together. |
I would keep them separate - I split them on purpose in that way to avoid having non-behavioural and behavioural changes in the same commit, following the guidelines of what I was asked to do for other PRs e.g. #6497 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I’m submitting part of my review so that you can start working while I finish the rest of it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is all my comments for rule.py
. Some may apply also to rule_grouping.py
Fixing the "NOT FOR 34.0.0rc2" comments from rucio#6516.
Follow-up for the "NOT FOR 34.0.0rc2" comments: #6575 |
Fixing the "NOT FOR 34.0.0rc2" comments from #6516.
Fixing the "NOT FOR 34.0.0rc2" comments from #6516.
* Rules: remove unnecessary arguments in __find_surplus_locks_and_remove_them * Rules: add type hints; rucio#6454 * Rules: update query to SQLAlchemy 2.0 syntax; rucio#6057 * Rules: Add RuleDict typed dict; rucio#6454
* Rules: remove unnecessary arguments in __find_surplus_locks_and_remove_them * Rules: add type hints; rucio#6454 * Rules: update query to SQLAlchemy 2.0 syntax; rucio#6057 * Rules: Add RuleDict typed dict; rucio#6454
Fixing the "NOT FOR 34.0.0rc2" comments from rucio#6516.
Adding type annotations to
rule.py
and some correlated type annotations torule_grouping.py
. Part of #6454.Tests pass locally.