Skip to content

docs: use the configured repository in the publishing example - #11125

Open
raghav-sharma-ai wants to merge 1 commit into
python-poetry:mainfrom
raghav-sharma-ai:patch-1
Open

raghav-sharma-ai wants to merge 1 commit into
python-poetry:mainfrom
raghav-sharma-ai:patch-1

Conversation

@raghav-sharma-ai

Copy link
Copy Markdown

The private-repository walkthrough configures repositories.foo, but the final publish command uses foo-pub, which is only configured in an optional note. Following the main steps can therefore fail with Repository foo-pub is not defined.

Use foo in the main command and explain when to substitute foo-pub.

Validation: checked the configuration sequence against the repository lookup in Publisher.publish; git diff --check passes. Documentation-only change; no package was uploaded and the full documentation build was not run.

Pull Request Check List

  • Tests for changed code: not applicable (no runtime code changed).
  • Updated documentation.

Prepared with AI assistance (OpenAI Codex).

Keep the main publishing command consistent with repositories.foo and explain the optional foo-pub configuration.

Prepared with AI assistance (OpenAI Codex).

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hey - I've reviewed your changes and they look great!

Sourcery assessment

Approved.


Sourcery is free for open source - if you like our reviews please consider sharing them ✨

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.

1 participant