-
-
Notifications
You must be signed in to change notification settings - Fork 14.5k
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
python312Packages.markitdown: init at 0-unstable-2024-12-18 #365681
Conversation
4b900d4
to
e12a099
Compare
e12a099
to
4c51d1e
Compare
3e37130
to
03b15e6
Compare
Running nixpkgs-review here: https://github.com/lucasew/nixcfg/actions/runs/12364289266/job/34507250833 |
|
03b15e6
to
35f19e8
Compare
|
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.
LGTM !
IMO the things left to be done are
|
It is already in there
I couldn't get cobble, precisely and funk tests running. Help might be needed, I tested edit: I just found out a way to get them running! |
35f19e8
to
5d5bd00
Compare
I'd suggest passing the additional test paths via pytest args
|
5d5bd00
to
5210962
Compare
Thank you, this is what I did. I couldn't enable tests everywhere because they are relying on |
Running nixpkgs-review here https://github.com/lucasew/nixcfg/actions/runs/12379680406/job/34554367173 |
|
Otherwise it seems all OK |
Should I remove it? edit: I removed it now. |
5210962
to
b10a644
Compare
Related issue about the nose thing #326513 |
Diff-wise LGTM Running final nixpkgs-review on https://github.com/lucasew/nixcfg/actions/runs/12382905002/job/34564584179 |
|
b10a644
to
c359044
Compare
c359044
to
3854473
Compare
@GaetanLepage I found a way to fix the tests ! all good. |
|
|
|
|
Oops, went to bed, feel free to fix it or I'll do it tomorrow. |
I can't
Tried checking out with |
3854473
to
8caab63
Compare
|
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.