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

chore: add basic md table with dollar sign test #89

Merged
merged 2 commits into from
Jan 18, 2024

Conversation

deer
Copy link
Contributor

@deer deer commented Jan 18, 2024

closes #65

@lionel-rowe @lino-levan, what do you think about this? I took the md from the issue and wrapped it with the boilerplate from the README.md file and everything is rendering correctly. Changing the spacing between the pipe and dollar sign had no effect.

So this PR is to just assert that the functionality works and won't break in the future.

@deer
Copy link
Contributor Author

deer commented Jan 18, 2024

@hashrock, since you're so fast with reviewing these, what do you think about this one?

@hashrock
Copy link
Contributor

hashrock commented Jan 18, 2024

Hmm, I'm not sure. The deps has been upgraded from marked@5.1.0 to marked@11.1.1 over the past few months, so it may have been fixed in the process.

Regardless, there's nothing more reliable than unit tests. If this issue still occurs, let's create another issue. Thank you for your contribution!

Copy link
Contributor

@hashrock hashrock left a comment

Choose a reason for hiding this comment

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

LGTM!

@hashrock hashrock merged commit 16a1681 into denoland:main Jan 18, 2024
1 check passed
@hashrock
Copy link
Contributor

@deer deer deleted the table_dollar_test branch January 18, 2024 14:23
@deer
Copy link
Contributor Author

deer commented Jan 18, 2024

A soft-conflict with the previous PR. I'll open a fix soon.

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.

Rendering bug with dollar signs in tables
2 participants