Skip to content

Commit

Permalink
Merge pull request #372 from Molrn/fix-uncaught-exceptions
Browse files Browse the repository at this point in the history
fix: exceptions not caught with `except Exception as e:`
  • Loading branch information
zainhoda authored Apr 21, 2024
2 parents aa78daf + 48293bf commit 8307958
Showing 1 changed file with 8 additions and 8 deletions.
16 changes: 8 additions & 8 deletions src/vanna/exceptions/__init__.py
Original file line number Diff line number Diff line change
@@ -1,46 +1,46 @@
class ImproperlyConfigured(BaseException):
class ImproperlyConfigured(Exception):
"""Raise for incorrect configuration."""

pass


class DependencyError(BaseException):
class DependencyError(Exception):
"""Raise for missing dependencies."""

pass


class ConnectionError(BaseException):
class ConnectionError(Exception):
"""Raise for connection"""

pass


class OTPCodeError(BaseException):
class OTPCodeError(Exception):
"""Raise for invalid otp or not able to send it"""

pass


class SQLRemoveError(BaseException):
class SQLRemoveError(Exception):
"""Raise when not able to remove SQL"""

pass


class ExecutionError(BaseException):
class ExecutionError(Exception):
"""Raise when not able to execute Code"""

pass


class ValidationError(BaseException):
class ValidationError(Exception):
"""Raise for validations"""

pass


class APIError(BaseException):
class APIError(Exception):
"""Raise for API errors"""

pass

0 comments on commit 8307958

Please sign in to comment.