Skip to content

Enhance exception routes not exists - #91

Merged
marcuxyz merged 5 commits into
mainfrom
test/enhance_exception_routes_not_exists
Jul 11, 2026
Merged

marcuxyz merged 5 commits into
mainfrom
test/enhance_exception_routes_not_exists

Conversation

@marcuxyz

Copy link
Copy Markdown
Owner

Custom exception message for the user if the user doesn't create routes.py.

@marcuxyz
marcuxyz force-pushed the test/enhance_exception_routes_not_exists branch from 5c3ff62 to 14ba991 Compare July 11, 2026 22:48

Copilot AI 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.

Pull request overview

This PR bumps the package version and introduces a custom exception when a user’s routes.py module cannot be imported, aiming to provide a clearer error message if routes are not defined.

Changes:

  • Bump version from 0.3.0 to 0.3.1 across package metadata and tests.
  • Add RoutesModuleNotFoundError and raise it from BlueprintBinding when importing {path}.routes fails.
  • Remove the legacy BlueprintMiddleware implementation.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
tests/test_version_functional.py Updates expected __version__ for the new release.
pyproject.toml Bumps project version to 0.3.1.
flask_mvc/version.py Bumps library __version__ to 0.3.1.
flask_mvc/core/exceptions.py Adds RoutesModuleNotFoundError for missing routes module scenarios.
flask_mvc/middlewares/blueprint_binding.py Wraps routes import to raise a clearer custom exception on missing routes module.
flask_mvc/middlewares/blueprint_middleware.py Removes the old blueprint middleware implementation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread flask_mvc/middlewares/blueprint_binding.py Outdated
Comment thread flask_mvc/middlewares/blueprint_binding.py Outdated
@marcuxyz
marcuxyz merged commit 215fc87 into main Jul 11, 2026
8 checks passed
@marcuxyz
marcuxyz deleted the test/enhance_exception_routes_not_exists branch July 11, 2026 23:07
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