Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updates and corrections #67

Merged
merged 5 commits into from
Sep 5, 2024
Merged

Updates and corrections #67

merged 5 commits into from
Sep 5, 2024

Conversation

AbstractUmbra
Copy link
Collaborator

Description

Updates some key features of the bot to reflect new goings on in the server and sister servers.

  • Clean up around dependencies

    • requirements files were outdated and unmaintained, so they have been removed
    • specify the speed extra of discord.py
    • update all dependencies to date
  • Webserver config entry

    • conditionally spawns the StarlettePlus app if the required config keys are present, and also allows setting the host and port arguments. Expansible to included others if needed.
  • Docker

    • fix warning on ENTRYPOINT directive for non-JSON based usage. (python run -O ... vs ["python", "run", "-O", "..."]).
    • re-order some keys in docker compose for sanity
    • remove deprecated version key in compose

Checklist

  • If code changes were made then they have been tested.
    • I have updated the documentation to reflect the changes.
    • I have updated the changelog with a quick recap of my changes.
  • This PR fixes an issue.
  • This PR adds something new (e.g. new method or parameters).
  • This PR is a breaking change (e.g. methods or parameters removed/renamed)
  • This PR is not a code change (e.g. documentation, README, ...)
  • I have read and agree to the Developer Certificate of Origin for this contribution

@AbstractUmbra AbstractUmbra self-assigned this Sep 5, 2024
@AbstractUmbra AbstractUmbra added the enhancement New feature or request label Sep 5, 2024
launcher.py Outdated Show resolved Hide resolved
@EvieePy EvieePy merged commit 3dd9195 into main Sep 5, 2024
2 checks passed
@EvieePy EvieePy deleted the feature/updates-and-corrections branch September 5, 2024 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants