Skip to content

docs: Run prettier-markdown on markdown files.#2664

Merged
toktok-releaser merged 1 commit into
TokTok:masterfrom
iphydf:mdformat
Feb 9, 2024
Merged

docs: Run prettier-markdown on markdown files.#2664
toktok-releaser merged 1 commit into
TokTok:masterfrom
iphydf:mdformat

Conversation

@iphydf

@iphydf iphydf commented Feb 9, 2024

Copy link
Copy Markdown
Member

This change is Reviewable

@iphydf iphydf added this to the v0.2.19 milestone Feb 9, 2024
@iphydf
iphydf marked this pull request as ready for review February 9, 2024 01:00
@iphydf
iphydf force-pushed the mdformat branch 2 times, most recently from 9dcc57a to 582615d Compare February 9, 2024 01:06
@codecov

codecov Bot commented Feb 9, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (969e3a2) 73.76% compared to head (8d0811a) 73.69%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2664      +/-   ##
==========================================
- Coverage   73.76%   73.69%   -0.07%     
==========================================
  Files         148      148              
  Lines       30479    30479              
==========================================
- Hits        22482    22462      -20     
- Misses       7997     8017      +20     

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

@nurupo nurupo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Despite what the commit title says. it looks like the docs/* files were at least partially manually formatted, as it's a bit hard to believe that a tool would do such edits. The resulting documents do look a lot better though, so LGTM.

@toktok-releaser
toktok-releaser merged commit 8d0811a into TokTok:master Feb 9, 2024
@iphydf
iphydf deleted the mdformat branch January 1, 2026 06:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants