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

Conversation

@Borda
Copy link
Contributor

@Borda Borda commented Jul 22, 2024

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?

let's update all notebooks if any requirement in the "book"/default has changed

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 Jul 22, 2024

Codecov Report

Attention: Patch coverage is 86.66667% with 2 lines in your changes missing coverage. Please review.

Project coverage is 70%. Comparing base (bc27060) to head (2d69604).

Additional details and impacted files
@@ Coverage Diff @@ ## main #337 +/- ## =================================== - Coverage 73% 70% -2%  =================================== Files 2 2 Lines 405 413 +8 =================================== - Hits 294 291 -3  - Misses 111 122 +11 
@Borda Borda marked this pull request as ready for review July 22, 2024 22:05
@Borda Borda requested a review from ethanwharris as a code owner July 22, 2024 22:05
@Borda Borda added the enhancement New feature or request label Jul 22, 2024
@Borda Borda merged commit e705ace into main Jul 22, 2024
@Borda Borda deleted the req/update branch July 22, 2024 23:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request topic: CI/CD

1 participant