Skip to content

Add more Pulp Exceptions.#439

Merged
mdellweg merged 1 commit intopulp:mainfrom
aKlimau:add-pulp-exceptions
Apr 23, 2026
Merged

Add more Pulp Exceptions.#439
mdellweg merged 1 commit intopulp:mainfrom
aKlimau:add-pulp-exceptions

Conversation

@aKlimau
Copy link
Copy Markdown
Contributor

@aKlimau aKlimau commented Apr 22, 2026

Assisted-by: Claude Sonnet 4.6 noreply@anthropic.com

📜 Checklist

  • Commits are cleanly separated with meaningful messages (simple features and bug fixes should be squashed to one commit)
  • A changelog entry or entries has been added for any significant changes
  • Follows the Pulp policy on AI Usage
  • (For new features) - User documentation and test coverage has been added

See: Pull Request Walkthrough

  Assisted-by: Claude Sonnet 4.6 <noreply@anthropic.com>
@aKlimau aKlimau force-pushed the add-pulp-exceptions branch from e103423 to 5049540 Compare April 22, 2026 13:31
Stage,
)

from pulp_gem.app.exceptions import RemoteConnectionError, RemoteURLRequiredError
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

These two look rather generic. Can they live in pulpcore.plugin?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It's probably ok to do that later...

@mdellweg mdellweg merged commit 99e3a0f into pulp:main Apr 23, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants