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

Markdown tables #2977

Merged
merged 12 commits into from May 31, 2023
Merged

Markdown tables #2977

merged 12 commits into from May 31, 2023

Conversation

darrenburns
Copy link
Member

@darrenburns darrenburns commented May 31, 2023

Type of changes

  • Bug fix
  • New feature
  • Documentation / docstrings
  • Tests
  • Other

Checklist

  • I've run the latest black with default args on new code.
  • I've updated CHANGELOG.md and CONTRIBUTORS.md where appropriate.
  • I've added tests for new code.
  • I accept that @willmcgugan may be pedantic in the code review.

Description

Adds support for Markdown tables.
Includes support for column alignment.

image

@codecov-commenter
Copy link

codecov-commenter commented May 31, 2023

Codecov Report

Patch coverage: 32.30% and project coverage change: -3.61 ⚠️

Comparison is base (51f70fa) 98.31% compared to head (5c8250a) 94.70%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2977      +/-   ##
==========================================
- Coverage   98.31%   94.70%   -3.61%     
==========================================
  Files          74       74              
  Lines        7949     8013      +64     
==========================================
- Hits         7815     7589     -226     
- Misses        134      424     +290     
Flag Coverage Δ
unittests 94.70% <32.30%> (-3.61%) ⬇️

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

Impacted Files Coverage Δ
rich/markdown.py 87.77% <32.30%> (-11.90%) ⬇️

... and 3 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@darrenburns darrenburns marked this pull request as ready for review May 31, 2023 13:51
@@ -4,6 +4,9 @@

from markdown_it import MarkdownIt
from markdown_it.token import Token
from typing_extensions import get_args

from rich.table import Table
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can we use relative imports to match the rest of the code base.

@willmcgugan willmcgugan merged commit ea78218 into master May 31, 2023
17 checks passed
@willmcgugan willmcgugan deleted the markdown-tables branch May 31, 2023 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants