Skip to content

fix : removed unused make_response import and duplicate has_request_context imports in errors/handlers.py - #1422

Open
tmdeveloper007 wants to merge 2 commits into
komalharshita:mainfrom
tmdeveloper007:pr-tm-1417
Open

fix : removed unused make_response import and duplicate has_request_context imports in errors/handlers.py#1422
tmdeveloper007 wants to merge 2 commits into
komalharshita:mainfrom
tmdeveloper007:pr-tm-1417

Conversation

@tmdeveloper007

Copy link
Copy Markdown
Contributor

Summary of What Has Been Done

In src/errors/handlers.py, two import issues were fixed:

  1. make_response was imported from Flask but never used - removed from the import line.
  2. has_request_context was imported twice (once in the group import, and then twice separately) - consolidated to a single occurrence in the group import.

Changes Made

  • Removed make_response from the Flask import line.
  • Removed the duplicate from flask import has_request_context, request line.
  • Removed the duplicate from flask import has_request_context line.
  • Kept has_request_context in the main Flask import line.

Impact it Made

  • Removes dead code and eliminates duplicate imports.
  • Follows Python import hygiene best practices.
  • Makes the file cleaner and easier to maintain.

Note: Please assign this PR to the tmdeveloper007 account.

Closes #1417

@vercel

vercel Bot commented Jul 27, 2026

Copy link
Copy Markdown

@tmdeveloper007 is attempting to deploy a commit to the komalsony234-1530's projects Team on Vercel.

A member of the Team first needs to authorize it.

@komalharshita

Copy link
Copy Markdown
Owner

@tmdeveloper007 kindly resolve the merge conflicts

@komalharshita komalharshita added type:bug Something isn't working need review Further information is requested level:beginner quality:clean bug labels Jul 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug gssoc-2026 level:beginner need review Further information is requested quality:clean type:bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix : remove unused make_response import and duplicate has_request_context import in errors/handlers.py

2 participants