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.

Sign up for GitHub

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

Jump to bottom

Module-ify and add some type hinting; move to async sqlite3 lib #572

Merged
merged 127 commits into from
Sep 7, 2024
Merged

Module-ify and add some type hinting; move to async sqlite3 lib #572

merged 127 commits into from
Sep 7, 2024

Conversation

davidlougheed
Copy link
Member

Description

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@davidlougheed davidlougheed changed the title [WIP] Modularize and add some type hinting; modernize a bit [WIP] Modularize and add some type hinting; modernize some code Sep 3, 2022
@davidlougheed
type hinting work
d785868
@davidlougheed
linting / code style / small fixes to strings
5b2f5d4
@davidlougheed
hinting
a85d893
@davidlougheed
add aiosqlite and mypy typings for (optional) typechecking
d898859
@davidlougheed
relocate config & remove configurable schema location
6a1a6ea
@davidlougheed
oop
745c20c
@davidlougheed
add new base cog which factors out some common functionality
2b1efcc 
use it in basic cogs that dont need additional changes
@davidlougheed
relocate schema
74452c0
@davidlougheed
add package manifest
1baa455
@davidlougheed
docs: update info on config
fd957ed
@davidlougheed
rewrite bot.py with async database + some code style changes
413491f
@davidlougheed
rewrite main.py with aiosqlite and code style changes
ac22e95
@davidlougheed
rewrite banner cog with aiosqlite and code style changes
e290cb0
@davidlougheed
rewrite currency cog with aiosqlite and code style changes
7513cec
@davidlougheed
rewrite customreactions cog with aiosqlite and refactoring
0d20ab1
@davidlougheed
rewrite games cog with new base cog
3223b0b
@davidlougheed
rewrite helpers cog and images cog with new base cog
fa3b42a
@davidlougheed
rewrite mod cog with aiosqlite and new base cog
79ca287
@davidlougheed
change music cog to use new base cog
e32f5ae
@davidlougheed
change quote cog to use aiosqlite and new base cog
bb068ab
davidlougheed and others added 28 commits June 3, 2023 09:30
@davidlougheed
fix: muted role checks
0c5d87b
@davidlougheed
fix: music config model default start vol value
178a8a1
@davidlougheed
chore: port legacy assignable roles config to new config model
8792524
@davidlougheed
ci: tweak formatting check
253f504
@davidlougheed
fix: music config access
1e16edd
@davidlougheed
chore: update discordpy to 1.7.3
3254f5c
@davidlougheed
fix: type hints causing bad conversions in currency cog
0d4a715
@davidlougheed
fix: type hints causing bad conversions in memes and roles cogs
d3ee5b4
@davidlougheed
chore: add log webhook env vars to example env
f4f5c2b
@davidlougheed
chore: remove old config class
7624fb2
@davidlougheed
fix: listing all roles not working
df41acc
@davidlougheed
example env wording
7cf6dda
@namemcguffin
[fix]: fixed field name in example.env
6bc59f7
@davidlougheed
update course year range to 24-25
69cda94
@davidlougheed
steal #519
0594656
@davidlougheed
update dependencies
55a261a
@davidlougheed
dockerfile work
4efd41c
@davidlougheed
some paginator type hinting
4c15035
@davidlougheed
fix add_quote full name
7681882
@davidlougheed
fix some member converters in currency/quotes
a1a59e0
@davidlougheed
custom reactions elifs
a07addd
@davidlougheed
lint: run black
63bca21
@davidlougheed
remove old config.ini
7c33822
@davidlougheed
docs: update config / deployment details
9d62e54
@davidlougheed
ci: update actions
bb156fc
@davidlougheed
chore: clear apt lists + combine apt RUN statements
a222928
@davidlougheed
chore(deps): update uvloop
5839f88
@davidlougheed
ci: build an edge (master-branch) image too
bceb4b5
@davidlougheed davidlougheed merged commit 253f736 into dev Sep 7, 2024
3 of 4 checks passed
@davidlougheed davidlougheed deleted the refact branch September 7, 2024 19:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@le-potate le-potate le-potate approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@davidlougheed @le-potate @namemcguffin