Skip to content
This repository was archived by the owner on Aug 28, 2025. It is now read-only.

Conversation

@rohitgr7
Copy link
Contributor

@rohitgr7 rohitgr7 commented Apr 7, 2022

Before submitting

  • Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
  • Did you make sure to update the docs?
  • Did you write any new necessary tests?

What does this PR do?

Same as Lightning-AI/pytorch-lightning#12542
currently failing on master.

PR review

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?

Make sure you had fun coding 🙃

@codecov
Copy link

codecov bot commented Apr 7, 2022

Codecov Report

Merging #153 (54c4ff1) into main (0b955b5) will not change coverage.
The diff coverage is n/a.

@@         Coverage Diff         @@
##           main   #153   +/-   ##
===================================
  Coverage    75%    75%           
===================================
  Files         2      2           
  Lines       353    353           
===================================
  Hits        265    265           
  Misses       88     88           

@Borda Borda merged commit 5235ab9 into main Apr 7, 2022
@Borda Borda deleted the ci/precommit_black branch April 7, 2022 11:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants