Skip to content

docs(embedder-tool): fix link to relocated MLXEmbedders library#486

Closed
x-n2o wants to merge 1 commit into
ml-explore:mainfrom
x-n2o:embedder-tool-readme-link
Closed

docs(embedder-tool): fix link to relocated MLXEmbedders library#486
x-n2o wants to merge 1 commit into
ml-explore:mainfrom
x-n2o:embedder-tool-readme-link

Conversation

@x-n2o

@x-n2o x-n2o commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Proposed changes

Fixes #483.

Tools/embedder-tool/README.md links to Libraries/Embedders/README.md, which 404s — the embedders library moved to the mlx-swift-lm package as MLXEmbedders. This updates the link to point there, matching the form already used in Applications/MLXChatExample/README.md.

Surfaced by the markdown link check proposed in #479.

Checklist

Put an x in the boxes that apply.

  • I have read the CONTRIBUTING document
  • I have run pre-commit run --all-files to format my code / installed pre-commit prior to committing changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have updated the necessary documentation (if needed)

@x-n2o

x-n2o commented Jun 11, 2026

Copy link
Copy Markdown
Contributor Author

Closing — the relocated library target is covered by the .check-links-ignore blocklist in #479 instead.

@x-n2o x-n2o closed this Jun 11, 2026
@x-n2o x-n2o deleted the embedder-tool-readme-link branch June 11, 2026 22: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.

[BUG] embedder-tool README links to missing Embedders documentation

1 participant