Open DeekshaMohanty opened 2 months ago
Check out this pull request on
See visual diffs & provide feedback on Jupyter Notebooks.
Powered by ReviewNB
*beep* *bop*
Hi, human.
I'm the @tardis-bot and couldn't find your records in my database. I think we don't know each other, or you changed your credentials recently.
Please add your name and email to .mailmap
in your current branch and push the changes to this pull request.
In case you need to map an existing alias, follow this example.
*beep* *bop*
Hi, human.
I'm the @tardis-bot and couldn't find your records in my database. I think we don't know each other, or you changed your credentials recently.
Please add your name and email to .mailmap
in your current branch and push the changes to this pull request.
In case you need to map an existing alias, follow this example.
*beep* *bop*
Hi, human.
I'm the @tardis-bot and couldn't find your records in my database. I think we don't know each other, or you changed your credentials recently.
Please add your name and email to .mailmap
in your current branch and push the changes to this pull request.
In case you need to map an existing alias, follow this example.
*beep* *bop*
Hi, human.
I'm the @tardis-bot and couldn't find your records in my database. I think we don't know each other, or you changed your credentials recently.
Please add your name and email to .mailmap
in your current branch and push the changes to this pull request.
In case you need to map an existing alias, follow this example.
*beep* *bop*
Hi, human.
I'm the @tardis-bot and couldn't find your records in my database. I think we don't know each other, or you changed your credentials recently.
Please add your name and email to .mailmap
in your current branch and push the changes to this pull request.
In case you need to map an existing alias, follow this example.
@DeekshaMohanty can you please fix the .mailmap
problem.
*beep* *bop* Hi human, I ran ruff on the latest commit (68bdb3a4b52e7486c2cd275bc9a9ae262516c402). Here are the outputs produced. Results can also be downloaded as artifacts here. Summarised output:
Complete output(might be large):
Points 6 and 7 for the issue linked to this PR need some further discussion. #2701
Attention: Patch coverage is 93.33333%
with 5 lines
in your changes missing coverage. Please review.
Project coverage is 70.87%. Comparing base (
f8f664c
) to head (68bdb3a
). Report is 86 commits behind head on master.
Files with missing lines | Patch % | Lines |
---|---|---|
tardis/io/logger/logger.py | 92.85% | 5 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
:pencil: Description
Fixes #2701
Type: :rocket:
feature
:vertical_traffic_light: Testing
How did you test these changes?
:ballot_box_with_check: Checklist
build_docs
label