Skip to content
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

Fix max table width calculation #289

Merged
merged 10 commits into from
Mar 28, 2024
Merged

Conversation

stuertz
Copy link
Contributor

@stuertz stuertz commented Mar 3, 2024

Try to fix max_table_width computation

fixes: #288

@hugovk hugovk added the changelog: Fixed For any bug fixes label Mar 3, 2024
Copy link

codecov bot commented Mar 3, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.48%. Comparing base (6498a57) to head (cd8f2b8).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #289      +/-   ##
==========================================
+ Coverage   94.38%   94.48%   +0.10%     
==========================================
  Files           5        5              
  Lines        2404     2431      +27     
==========================================
+ Hits         2269     2297      +28     
+ Misses        135      134       -1     
Flag Coverage Δ
macos-latest 94.44% <100.00%> (+0.10%) ⬆️
ubuntu-latest 94.44% <100.00%> (+0.10%) ⬆️
windows-latest 94.40% <100.00%> (+0.10%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@stuertz
Copy link
Contributor Author

stuertz commented Mar 10, 2024

Hi @hugovk is something missing here? Are you waiting for something to be added?

@hugovk hugovk changed the title 288 max table width Fix max table width calculation Mar 13, 2024
@hugovk
Copy link
Member

hugovk commented Mar 13, 2024

Hi! Thank you very much for the PR! I'm just waiting to find time to review it :) I'll definitely try to get to it this month.

Copy link
Member

@hugovk hugovk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for this! This is mainly style/readability suggestions.

src/prettytable/prettytable.py Outdated Show resolved Hide resolved
src/prettytable/prettytable.py Outdated Show resolved Hide resolved
src/prettytable/prettytable.py Outdated Show resolved Hide resolved
stuertz and others added 3 commits March 27, 2024 15:01
@hugovk hugovk merged commit 39ea810 into jazzband:main Mar 28, 2024
27 checks passed
@hugovk
Copy link
Member

hugovk commented Mar 28, 2024

Thank you very much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog: Fixed For any bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

max_table_width does not work as expected
2 participants