Clean up dependencies

[tianyizheng02]

As a follow-up to #176, I want to clean up the dependencies listed in the Pipfile and requirements.txt. Some listed dependencies are no longer used and should be deleted, while others are purely dev dependencies (i.e., used by us for development purposes and not actually required by PittAPI).

I'm opening an issue for this because there are some dependencies that I'm not entirely sure of and I need help figuring out what to do about those ones; I've marked such dependencies with TODO.

---

KEY:

• DEV: Dev dependency, will be moved to [dev-packages] in the Pipfile
• TODO: I have a question and/or need clarification about this dependency

• REMOVE: Not actually used as a dependency, marking for deletion

These are the dependencies currently listed in the Pipfile:

• beautifulsoup4 = "*": REMOVE
  • We've already phased out BeautifulSoup in favor of requests and other packages.
• coverage = "==7.4.3": TODO, DEV
  • Re: 8d9ab9331a6a8f8c2f67d54cf34275b711f75c95, is there any reason why we must have v7.4.3? It should be fine to use the newest version.
• requests = "*"
• responses = "*"
• grequests = "*"
• lxml = "*": REMOVE
  • This needs to be replaced with lxml_html_clean: we only had lxml because we needed it as an optional dependency of requests-html, but the required functionality has been broken off into a separate package. This is what currently happens if you try to run the tests in people_test.py:

    ...
    E   ImportError: lxml.html.clean module is now a separate project lxml_html_clean.
    E   Install lxml[html_clean] or lxml_html_clean directly.

• nose = "*": REMOVE
  • We replaced nose with pytest in #144.
• nose-cov = "*": REMOVE
  • See nose
• nose-timer = "*": REMOVE
  • See nose
• typing = "*"
• virtualenv = "*": TODO REMOVE
  • I don't think we directly use this for anything; can someone confirm? Not used, should be safe to remove.
• black = "*": DEV
• sphinx = "*": TODO
  • I believe this is meant for generating PittAPI's docs (see c8663b9882000edb388efe3e553df5ec76e328ff), but we haven't really done anything about that yet. Should we just mark it as a dev dependency?
• requests-html = "*"
• parse = "*": REMOVE
  • We no longer use parse anywhere in the repo, and I think it was phased out along with BeautifulSoup.
• pre-commit = "*": DEV
• pytest = "*": DEV
• pytest-cov = "==4.1.0": TODO, DEV
  • Re: 8d9ab9331a6a8f8c2f67d54cf34275b711f75c95, is there any reason why we must have v4.1.0? It should be fine to use the newest version.
• pytest-xdist = "*": TODO
  • This pytest plugin allows for testing using multiple cores, but we never use it. Should we simply remove this dependency? Or should we actually use it by using it in our GitHub Actions workflows?
• flake8 = "*": DEV

[tianyizheng02]

@nij-patel, you were the one who did most of the work in upgrading the repo to Python 3.12, and @FishOfPitt116, you were the one who updated the versions for coverage and pytest-cov, so I'd appreciate it if you two could take a look at my TODOs.