Skip to content

Remove unused asyncpg dependency - #512

Merged
cigamit merged 1 commit into
ctrliq:mainfrom
blaipr:remove/asyncpg
Jun 25, 2026
Merged

Remove unused asyncpg dependency#512
cigamit merged 1 commit into
ctrliq:mainfrom
blaipr:remove/asyncpg

Conversation

@blaipr

@blaipr blaipr commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

SUMMARY

Remove asyncpg, an asyncio PostgreSQL driver that is never used.

  • AWX uses psycopg (sync) for its Django database backend, not asyncpg
  • Zero imports anywhere in the codebase
  • pip show asyncpg reports Required-by: empty - no reverse deps
  • Full Python test suite passes with it uninstalled (3,503 passed)
  • Removes requirement from requirements.in, requirements.txt, and licenses/

ISSUE TYPE

  • Bug, Docs Fix or other nominal change

COMPONENT NAME

  • Other

ASCENDER VERSION

awx: 25.4.1.dev

ADDITIONAL INFORMATION

Identified by auditing all pinned Python production dependencies for actual usage.

asyncpg is an asyncio PostgreSQL driver that is never imported
anywhere in the codebase. AWX uses psycopg (sync) for its Django
database backend, not asyncpg. No other installed package depends
on it.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 removes the unused asyncpg dependency from Ascender’s pinned Python dependency set, aligning production requirements with the actual database stack (Django + synchronous psycopg).

Changes:

  • Removed asyncpg from requirements/requirements.in.
  • Dropped the pinned asyncpg==0.30.0 entry from requirements/requirements.txt.
  • Removed the corresponding third-party license file (licenses/asyncpg.txt).

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
requirements/requirements.txt Removes the pinned asyncpg package entry from the compiled requirements.
requirements/requirements.in Removes asyncpg from the source requirements list so it won’t be reintroduced on recompile.
licenses/asyncpg.txt Removes the license artifact for a dependency that is no longer shipped.

@cigamit
cigamit merged commit 1d9be97 into ctrliq:main Jun 25, 2026
@cigamit cigamit self-assigned this Jun 25, 2026
@cigamit cigamit added javascript Pull requests that update javascript code dependencies Pull requests that update a dependency file and removed javascript Pull requests that update javascript code labels Jun 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Development

Successfully merging this pull request may close these issues.

3 participants