Skip to content

Comments

Trim down the requirements.txt file#31

Merged
spottseng merged 1 commit intomainfrom
clean-up
Feb 16, 2025
Merged

Trim down the requirements.txt file#31
spottseng merged 1 commit intomainfrom
clean-up

Conversation

@spottseng
Copy link
Owner

@spottseng spottseng commented Feb 16, 2025

Summary

Provide a brief summary of the changes in this PR and link to the relevant issue(s) if applicable.
There were a bunch of things listed in the requirements.txt file that were not necessary. This cleans up the file so it has only the necessary installs.

This also comments out the API call to DbIpCity. This doesn't work super well anyway since there is a rate limit and it's incredibly slow, so it just needs to be done a different way.

Current Behavior

Please describe the current behavior that you are modifying.
The requirements.txt file has a bunch of extra dependencies.

New Behavior

Please describe the behavior or changes that are being added by this PR.
The requirements.txt file only contains the dependencies it needs.

Does this new change introduce a breaking change?

  • Yes
  • No

If this introduces a breaking change:

  1. Describe the impact on the application below.
  2. Update the BREAKING.md file with the breaking change.
  3. Add "BREAKING CHANGE: [...]" to the commit description when merging.
  4. Create a ticket on the GitHub Project board to address the issue later.

Other Information

Any other information that is useful to note: visual changes, dependency changes, etc.

@spottseng spottseng merged commit d8d968f into main Feb 16, 2025
1 check passed
@spottseng spottseng deleted the clean-up branch February 16, 2025 14:26
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.

1 participant