Skip to content

Remove unused code and dependencies - #4934

Merged
bjester merged 3 commits into
learningequality:unstablefrom
rtibbles:plague_of_locusts
Mar 3, 2025
Merged

Remove unused code and dependencies#4934
bjester merged 3 commits into
learningequality:unstablefrom
rtibbles:plague_of_locusts

Conversation

@rtibbles

Copy link
Copy Markdown
Member

Summary

  • Cleans up unused profiling code
  • Removes associated dependencies
  • Cleans up documentation of this profiling code
  • Removes the unused django-mathfilters dependency
  • Removes all references to the no longer needed future library (as we no longer support or use Python 2.7) and removes it as a dependency

References

Inspired by numerous dependabot upgrade PRs for dependencies we do not need

Reviewer guidance

I think as long as tests pass this should be OK!

Clean up resultant unused dependencies.
Clean up outdated/unused pytest plugins.
@rtibbles

Copy link
Copy Markdown
MemberAuthor

D'oh!

Remove use of builtins imports.
Fix linting errors from commit.

@bjesterbjester left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - hopefully no surprises

@bjester
bjester merged commit 11ad113 into learningequality:unstableMar 3, 2025
@marcellamakimarcellamaki mentioned this pull request Mar 4, 2025
Sign up for freeto 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

@rtibbles@bjester