Skip to content

[codex] Add Hugging Face hosted links to README#43

Draft
kiankyars wants to merge 1 commit intohuggingface:mainfrom
kiankyars:codex/add-hf-hub-links
Draft

[codex] Add Hugging Face hosted links to README#43
kiankyars wants to merge 1 commit intohuggingface:mainfrom
kiankyars:codex/add-hf-hub-links

Conversation

@kiankyars
Copy link
Copy Markdown

Summary

Add a short top-level section to the README that points people to the hosted Hugging Face entry points for ML Intern.

What Changed

  • added a Try It on Hugging Face section near the top of the README
  • linked the ML intern explorers org
  • linked the hosted smolagents/ml-intern Space

Why

The repo README explained the local CLI flow, but it did not surface the main hosted Hub entry points that Hugging Face is promoting publicly.

Validation

  • docs-only change
  • no tests run

@akseljoonas
Copy link
Copy Markdown
Collaborator

@claude please review this PR. Focus on correctness, security, LiteLLM/Bedrock routing impact, and whether it's safe to merge against current main. Keep it concise and prefer inline comments where it matters.

@claude
Copy link
Copy Markdown

claude Bot commented Apr 24, 2026

Claude finished @akseljoonas's task in 46s —— View job


PR Review

  • Read CLAUDE.md for repo guidelines
  • Read README.md diff
  • Review correctness, security, routing impact, merge safety
  • Post final review

Verdict: ✅ Safe to merge — docs-only, no code changes.

Correctness

The added section (README.md lines 9–14) is straightforward and accurate per the PR description. Two minor nits:

  1. URL validity — neither https://huggingface.co/ml-agent-explorers nor https://huggingface.co/spaces/smolagents/ml-intern can be verified from this review. A broken link here is the first thing a new user sees. Please confirm both are publicly accessible before merging.

  2. Capitalization inconsistency — display text says ML intern explorers (lowercase i), but the project is consistently styled ML Intern (capital I) everywhere else in the README. Fix this →

Security

None. No credentials, tokens, or secrets introduced. Pure markdown.

LiteLLM / Bedrock routing impact

None. Only README.md is touched — zero routing code changes.

Merge safety against current main

Clean. The diff applies to the top of the file with no overlap with any in-flight code changes. No merge conflicts expected.


Pre-merge checklist (for the author):

  • Confirm huggingface.co/ml-agent-explorers is publicly accessible
  • Confirm huggingface.co/spaces/smolagents/ml-intern is publicly accessible
  • Optionally fix the ML internML Intern capitalization

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.

2 participants