Skip to content

Remove unused json-log-formatter dependency - #513

Merged
cigamit merged 1 commit into
ctrliq:mainfrom
blaipr:remove/json-log-formatter
Jun 25, 2026
Merged

Remove unused json-log-formatter dependency#513
cigamit merged 1 commit into
ctrliq:mainfrom
blaipr:remove/json-log-formatter

Conversation

@blaipr

@blaipr blaipr commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

SUMMARY

Remove json-log-formatter, a JSON logging formatter that is never used.

  • Zero imports in code, settings, or logging configuration
  • Not referenced in any LOGGING dict or log handler config
  • pip show json-log-formatter 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.

json-log-formatter is never imported or referenced in any code,
settings, or logging configuration. 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

Removes the unused json-log-formatter Python dependency from Ascender’s pinned dependency set, keeping production requirements and license artifacts aligned with actual runtime usage.

Changes:

  • Dropped json-log-formatter from requirements/requirements.in.
  • Removed the pinned entry from requirements/requirements.txt.
  • Deleted the corresponding licenses/JSON-log-formatter.txt artifact.

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 json-log-formatter==0.5.2 entry.
requirements/requirements.in Removes the source requirement (JSON-log-formatter) so it won’t be reintroduced on regen.
licenses/JSON-log-formatter.txt Removes the now-unneeded license file for the removed dependency.

@cigamit cigamit self-assigned this Jun 25, 2026
@cigamit cigamit added the dependencies Pull requests that update a dependency file label Jun 25, 2026
@cigamit
cigamit merged commit 5282891 into ctrliq:main 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