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
DM-35971: Pytest requires flake<5 #200
Conversation
Codecov Report
@@ Coverage Diff @@
## main #200 +/- ##
==========================================
- Coverage 82.46% 82.42% -0.05%
==========================================
Files 47 47
Lines 3753 3755 +2
Branches 675 676 +1
==========================================
Hits 3095 3095
- Misses 480 481 +1
- Partials 178 179 +1
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
bcd760b
to
47e9ab6
Compare
fix mypy
47e9ab6
to
17b103c
Compare
@@ -68,8 +68,10 @@ def mockDatasetTypeName(cls, datasetTypeName: str) -> str: | |||
"""Make mock dataset type name from actual dataset type name.""" | |||
return "_mock_" + datasetTypeName | |||
|
|||
def _get(self, ref: Union[DeferredDatasetRef, DatasetRef]) -> Any: | |||
def _get(self, ref: Optional[Union[DeferredDatasetRef, DatasetRef]]) -> Any: |
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.
Yes, I had that fix and the other fix on this ticket on #199
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.
Ah OK, then I take this package out of the ticket.
This will be fixed in #199 |
Sorry. I didn't mean to imply to close this becasue my PR is not going to merge for a few days and I think we need this PR merged before the weekly goes out. |
Checklist
doc/changes