Skip to content

fix(discovery): run the README commands from the Discovery directory - #197

Open
rudycelekli wants to merge 1 commit into
uber:mainfrom
rudycelekli:fix/discovery-readme-run-directory
Open

rudycelekli wants to merge 1 commit into
uber:mainfrom
rudycelekli:fix/discovery-readme-run-directory

Conversation

@rudycelekli

@rudycelekli rudycelekli commented Oct 19, 2026 •

Copy link
Copy Markdown

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update

Related issue: N/A (found by reading the code; reproduced on main at 6b33dff)

What changed?
Say "From the Discovery directory".

Why?
Discovery/adr_discovery/README.md:33 says "From the repository root:" then uv run adr-discovery ...; at the repo root uv fails: error: Failed to spawn: adr-discovery.

How did you test it?

Potential risks
The change only affects the input described above; other paths go through the same code as before, and the component's existing tests pass.

adr_discovery/README.md said to run `uv run adr-discovery` from the
repository root, where there is no pyproject.toml and uv fails with
"Failed to spawn: adr-discovery". The script is defined in
Discovery/pyproject.toml, as Discovery/README.md already says.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

This branch has not been deployed

No deployments
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