-
-
Notifications
You must be signed in to change notification settings - Fork 14.1k
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.terminaltables3: init at 4.0.0 #353928
base: master
Are you sure you want to change the base?
Conversation
34b6121
to
e2cd761
Compare
description = "Display simple tables in terminals"; | ||
homepage = "https://github.com/matthewdeanmartin/terminaltables3"; | ||
license = licenses.mit; | ||
maintainers = with maintainers; [ kekschen ]; |
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.
Also add yourself to maintainer-list.nix :)
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.
Already done. I'm still waiting for #353536
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.
I extracted it into #354251
This package is a python library, so please change your commit message to
|
e2cd761
to
69f599f
Compare
pythonImportsCheck = [ "terminaltables3" ]; | ||
|
||
# doesn't have any tests | ||
doCheck = false; |
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.
Uhhhm, there are tests: https://github.com/matthewdeanmartin/terminaltables3/tree/master/tests
It can be that the tests aren't included on PyPi. But they should be when you change src
to the GitHub-Repo.
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.
good idea, i'm gonna do that then.
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.
I don't think the fork cares about the tests
============================= test session starts ==============================
platform linux -- Python 3.12.7, pytest-8.3.3, pluggy-1.5.0
rootdir: /build/source
configfile: pyproject.toml
collected 801 items
tests/test_all_tables_e2e/test_ascii_table.py ..s [ 0%]
tests/test_all_tables_e2e/test_double_table.py ..s [ 0%]
tests/test_all_tables_e2e/test_github_table.py .. [ 0%]
tests/test_all_tables_e2e/test_porcelain_table.py .. [ 1%]
tests/test_all_tables_e2e/test_single_table.py .. [ 1%]
tests/test_all_tables_e2e/test_single_table_windows.py sss [ 1%]
tests/test_ascii_table.py ...................... [ 4%]
tests/test_base_table/test_gen_row_lines.py ............... [ 6%]
tests/test_base_table/test_gen_table.py .................... [ 8%]
tests/test_base_table/test_horizontal_border.py ................ [ 10%]
tests/test_base_table/test_table.py .......F [ 11%]
tests/test_build/test_build_border.py .................................. [ 16%]
........................................................................ [ 25%]
........................................................................ [ 34%]
........................................................................ [ 43%]
........................................................................ [ 52%]
........................................................................ [ 61%]
...................................FFFFFFFFFFFFFFFFFF.FFFFFFFFFFFFFF.FFF [ 70%]
FFFFFFFFFFFFFFF.FFFFFFFFFFFFFF..... [ 74%]
tests/test_build/test_build_row.py ...... [ 75%]
tests/test_build/test_combine.py ...... [ 76%]
tests/test_build/test_flatten.py .. [ 76%]
tests/test_examples.py ... [ 76%]
tests/test_terminal_io/test_get_console_info.py . [ 76%]
tests/test_terminal_io/test_set_terminal_title.py sssssssss [ 77%]
tests/test_terminal_io/test_terminal_size.py ..... [ 78%]
tests/test_width_and_alignment/test_align_and_pad_cell.py ........F..... [ 80%]
......F...........F...........F...........F......F......F......F......F. [ 89%]
.....F................................F [ 94%]
tests/test_width_and_alignment/test_column_max_width.py ... [ 94%]
tests/test_width_and_alignment/test_max_dimensions.py ........... [ 95%]
tests/test_width_and_alignment/test_table_width.py ... [ 96%]
tests/test_width_and_alignment/test_visible_width.py ................... [ 98%]
........... [100%]
=================================== FAILURES ===================================
__________________________________ test_color __________________________________
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.
A sample:
if left and right:
expected = left + expected + right
actual = build_border(outer_widths, "-", left, intersect, right, title=title)
> assert "".join(actual) == expected
E AssertionError: assert 'TEST--------' == '\x1b[34mTEST\x1b[0m--------'
E
E - TEST--------
E + TEST--------
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.
Very weird. I can replicate the failing tests. But I wonder why nearly every test got touched in the 4.0.0
commit matthewdeanmartin/terminaltables3@f1c465b
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.
looks like an autoformatting commit
69f599f
to
0000a5d
Compare
Things done
terminaltables3 package as discussed in #353771.
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.