Skip to content

feat: introduce ciel-specific Python exceptions - #137

Open
Gitjay11 wants to merge 1 commit into
fossi-foundation:mainfrom
Gitjay11:feat/ciel-specific-exceptions
Open

feat: introduce ciel-specific Python exceptions #137
Gitjay11 wants to merge 1 commit into
fossi-foundation:mainfrom
Gitjay11:feat/ciel-specific-exceptions

Conversation

@Gitjay11

Copy link
Copy Markdown

closes #131

Summary

This PR addresses issue #131 by introducing dedicated exception classes for ciel. Previously, ciel raised generic standard library exceptions (e.g., ValueError, RuntimeError, TypeError, IOError), forcing callers to catch broad exceptions and inadvertently capturing errors from third-party libraries or Python built-ins.

All new exception classes inherit from their corresponding standard library exceptions (ValueError, RuntimeError, TypeError, IOError) to ensure backward compatibility for callers catching base types while allowing fine-grained exception handling for ciel-specific errors.

Verification

  • Confirmed all 10 new exception classes correctly inherit from their respective base classes.
  • Verified top-level package imports (import ciel; ciel.InvalidPDKError, etc.).
  • Ensured backward compatibility alias (VersionNotFound is VersionNotFoundError) functions as expected.

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.

Introduce ciel-specific Python exceptions

1 participant