| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (7)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists. You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. ❤️ ShareComment @coderabbitai help to get the list of available commands and usage tips. |
Sorry, something went wrong.
|
Sorry about the repeat PR. I tried to do some trickery with rebasing, but am just going to recommit on the main branch |
Sorry, something went wrong.
|
I think also for the sake of less spam, I'll do two-three libraries at a time (Let me know if I shouldn't, and instead should have individual PR's for each library) |
Sorry, something went wrong.
|
@ShaharNaveh there is one test that is failing locally, but succeeding in the CI testing (test.test_enum.OldTestIntFlag.test_global_enum_str). Can you check it on your side, and advise? |
Sorry, something went wrong.
|
I believe you can use the following command to test your changes locally: # this runs all of the tests (not necessary if you know which test is affected)
cargo run --release --features encodings,sqlite -- -m test -j 1 -u all --slowest --fail-env-changed -v
# this runs only the test suite named "test_whatever" (usually located at `Lib/test/test_whatever.py`)
cargo run --release --features encodings,sqlite -- -m test -j 1 -u all --slowest --fail-env-changed -v test_whateverFor a list of all resources and more command-line options, you can execute cargo run --release --features encodings,sqlite -- -m test -h. |
Sorry, something went wrong.
Ah, thanks for the suggestion, will try the command you sent! |
Sorry, something went wrong.
There was a problem hiding this comment.
👍 Thank you!
you don't have to worry about spamming too much, but this way will be easier because tests are related to each others. In this PR, test_ssl and test_uuid is unexpectedly related to (probably) enum
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
No description provided.