Closed adelavega closed 1 year ago
thanks a lot @adelavega ! sorry for the late reply. could you run make format
or black
or merge the PR I opened on your branch just to make flake8 happy? thanks!
the macos failure looks like an issue with the pandas installation, hopefully it won't show up again
actually as it's just automatic formatting I just pushed it directly
Merging #34 (28c0254) into main (11ec73e) will not change coverage. The diff coverage is
100.00%
.
@@ Coverage Diff @@
## main #34 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 25 25
Lines 2024 2025 +1
=========================================
+ Hits 2024 2025 +1
Impacted Files | Coverage Δ | |
---|---|---|
src/pubget/_links.py | 100.00% <100.00%> (ø) |
thanks a lot @adelavega !! merging
Thank you Jerome!
On Tue, Jul 4, 2023, 11:45 AM Jérôme Dockès @.***> wrote:
Merged #34 https://github.com/neuroquery/pubget/pull/34 into main.
— Reply to this email directly, view it on GitHub https://github.com/neuroquery/pubget/pull/34#event-9723287184, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAVFLMES2PKDZRNVXCMNGO3XOQ3BFANCNFSM6AAAAAAZPCX5WY . You are receiving this because you were mentioned.Message ID: @.***>
Fixes #33
uri
) if this no type found (which happens with the uri tag)