Skip to content

fix (docs): correct typos#1622

Open
kimtth wants to merge 5 commits into
microsoft:mainfrom
kimtth:main
Open

fix (docs): correct typos#1622
kimtth wants to merge 5 commits into
microsoft:mainfrom
kimtth:main

Conversation

@kimtth

@kimtth kimtth commented Mar 18, 2026

Copy link
Copy Markdown

Summary

Fix typos, grammatical errors, and minor code issues across comments, docstrings, and error messages. No behavioral changes.

Changes

Typo fixes

File Line Before After
_base_converter.py 54 steam_info.url stream_info.url
_base_converter.py 74, 95 (mimetype, extension, charset, set) (mimetype, extension, charset)
_markitdown.py 583 # Add legaxy kwargs # Add legacy kwargs
_markitdown.py 634 User register_converter Use register_converter
_outlook_msg_converter.py 57 # Brue force # Brute force
_exceptions.py 43 Represents an a single attempt Represents a single attempt
_exceptions.py 72 provided no execution info provided no exception info
_docx_converter.py 33 (e.g.m headings) (e.g., headings)
_epub_converter.py 28 (e.g.m headings) (e.g., headings)
_transcribe_audio.py 27 MarkItdown MarkItDown
_doc_intel_converter.py 210 not availiable for not available for
test_cli_vectors.py 30 require mockig the url require mocking the url
test_cli_vectors.py 99 CLI readds from stdin CLI reads from stdin
test_module_misc.py 129, 146 the targted attribute the targeted attribute
test_sample_plugin.py 16 converter dirctly converter directly

Grammar fixes

File Line Before After
_markitdown.py 250 Plugins converters are already enabled Plugin converters are already enabled
_plugin.py (sample) 36 Converts an RTF file to in the simplest Converts an RTF file in the simplest
_plugin.py (sample) 63 an str using hte provided a str using the provided

Inconsistency fixes

File Line Before After
_markitdown.py 653 PRIORITY_SPECIFIC_FILE_FORMAT (== 10) PRIORITY_GENERIC_FILE_FORMAT (== 10)
__main__.py 104 --use-plugin option --use-plugins option
_exceptions.py 72 double space after dash in error msg single space (consistent with next line)

Code cleanup

File Description
_plain_text_converter.py Removed unused mammoth import and _dependency_exc_info (copy-paste artifact from docx converter)
_pptx_converter_with_ocr.py Removed duplicate from typing import BinaryIO, Any, Optional line

Files changed (15)

  • packages/markitdown-ocr/src/markitdown_ocr/_pptx_converter_with_ocr.py
  • packages/markitdown-sample-plugin/src/markitdown_sample_plugin/_plugin.py
  • packages/markitdown-sample-plugin/tests/test_sample_plugin.py
  • packages/markitdown/src/markitdown/__main__.py
  • packages/markitdown/src/markitdown/_base_converter.py
  • packages/markitdown/src/markitdown/_exceptions.py
  • packages/markitdown/src/markitdown/_markitdown.py
  • packages/markitdown/src/markitdown/converters/_doc_intel_converter.py
  • packages/markitdown/src/markitdown/converters/_docx_converter.py
  • packages/markitdown/src/markitdown/converters/_epub_converter.py
  • packages/markitdown/src/markitdown/converters/_outlook_msg_converter.py
  • packages/markitdown/src/markitdown/converters/_plain_text_converter.py
  • packages/markitdown/src/markitdown/converters/_transcribe_audio.py
  • packages/markitdown/tests/test_cli_vectors.py
  • packages/markitdown/tests/test_module_misc.py

@kimtth

kimtth commented May 25, 2026

Copy link
Copy Markdown
Author

Hi @afourney, following up on this PR. Let me know if any changes are needed on my end. Thanks.

@kimtth

kimtth commented May 29, 2026

Copy link
Copy Markdown
Author

@afourney
If you are no longer managing this repository, who is the appropriate reviewer? Could you please delegate this request to them?

I don’t see any reason not to accept this pull request, since the typo fixes are straightforward and clear.

@kimtth

kimtth commented Jun 2, 2026

Copy link
Copy Markdown
Author

@afourney
@yungshinlintw
@fafuxa

summary updated.
you can use github copilot app if you'd like to automate stacked pull requests.

@kimtth

kimtth commented Jun 4, 2026

Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree

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.

1 participant