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

feat/improve configuration #75

Merged
merged 80 commits into from
Apr 21, 2022
Merged

feat/improve configuration #75

merged 80 commits into from
Apr 21, 2022

Commits on Oct 4, 2021

  1. feat: rewrite the entire configuration system to use marshmallow and … 

    …attrs
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    b5c5417 View commit details
    Browse the repository at this point in the history

  2. config: remove config_default.toml since its now part of the classes 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    6a1b0c2 View commit details
    Browse the repository at this point in the history

  3. fix: add missing items to the configuration 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    5afc196 View commit details
    Browse the repository at this point in the history

  4. fix: have a test token in the pytest env 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    f8fabf5 View commit details
    Browse the repository at this point in the history

  5. enable slots on cfg classes, fix colour converter, properly prevent t… 

    …oken from being changed

Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    6cc4e20 View commit details
    Browse the repository at this point in the history

  6. fix: remove unused class definitions 

    when converting to a marshmallow schema, desert does not use class defined Meta classes

Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    82b1c73 View commit details
    Browse the repository at this point in the history

  7. config: add autogenerated default cfg toml file 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    4a80406 View commit details
    Browse the repository at this point in the history

  8. fix: properly deserialize discord.Colour to a string representation 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    311ff54 View commit details
    Browse the repository at this point in the history

  9. feat: add configuration via a yaml cfg file 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    62aa7dc View commit details
    Browse the repository at this point in the history

  10. fix: allow log level 50, as documented 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    e4d9e1e View commit details
    Browse the repository at this point in the history

  11. feat: made testable config system by refactor config file loading 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    70128af View commit details
    Browse the repository at this point in the history

  12. tools: add pre-commit hook to automatically export default config 

    if any modmail configuration file is edited, a pre-commit hook will run to check that the autogenerated files are up-to-date

Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    4707d6c View commit details
    Browse the repository at this point in the history

  13. tools: automatically export required environment varaibles to .env.te… 

    …mplate

Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    26cc817 View commit details
    Browse the repository at this point in the history

  14. tools: add app.json to configuration exports 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    b8d7080 View commit details
    Browse the repository at this point in the history

  15. minor: manaually update references to configuration files 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    19ddc22 View commit details
    Browse the repository at this point in the history

  16. fix: add missing modmail dependencies to additional hook dependencies 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    06aad80 View commit details
    Browse the repository at this point in the history

  17. fix: no configuration files now loads the default config 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    cc50262 View commit details
    Browse the repository at this point in the history

  18. fix: create a Config instance when adding it to the bot 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    bdc7d42 View commit details
    Browse the repository at this point in the history

  19. fix: load prefix from modmail_config.toml if not in .env 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    29de36e View commit details
    Browse the repository at this point in the history

  20. fix: ignore extra configuration variables 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    2166f8c View commit details
    Browse the repository at this point in the history

  21. minor: use attr helper method instead of reimplementing it 

    checked with a debug command while in a debug session
'{x for x in attr.fields(klass)} == attribs' is True.
They are exactly the same, so I've switched to using the attr method instead of rolling my own.

Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    be6b698 View commit details
    Browse the repository at this point in the history

  22. chore: switch _load_config to pack positional args rather than requir… 

    …e an iterable

Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    4a367c4 View commit details
    Browse the repository at this point in the history

  23. fix: rewrite environment variable parser to support all configuration… 

    … options

Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    4e085d7 View commit details
    Browse the repository at this point in the history

  24. feat: add ConfigMetadata part 1 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    4495938 View commit details
    Browse the repository at this point in the history

  25. fix recursive dict update overriding set values 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    0085f2a View commit details
    Browse the repository at this point in the history

  26. chore: update export docstring 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    9417d3c View commit details
    Browse the repository at this point in the history

  27. fix: respect priorty of configuration sources 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    bb1d956 View commit details
    Browse the repository at this point in the history

  28. fix pre-commit maybe 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    d5f1290 View commit details
    Browse the repository at this point in the history

  29. nit: rename config class Bot to BotCfg to prevent future confusion 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    f3c3326 View commit details
    Browse the repository at this point in the history

  30. fix: add ConfigMetadata part 2 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    6e14be0 View commit details
    Browse the repository at this point in the history

  31. chore: remove unused dict manipulation method 

    removed the now defunct _recursive_dict_update method

Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    c05e0bb View commit details
    Browse the repository at this point in the history

  32. chore: implement new metadata options on cfg default export 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    444aca1 View commit details
    Browse the repository at this point in the history

  33. tests: add config loading tests 

    test if the cofngiruation properly reads from provided files

Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    014e9aa View commit details
    Browse the repository at this point in the history

  34. feat: add configuration manager cog 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    2a2b9dc View commit details
    Browse the repository at this point in the history

  35. migrate some existing values to new configuration 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    9eeee2e View commit details
    Browse the repository at this point in the history

  36. minor: hide frozen values which can't be changed from the configuration 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    d6cbf86 View commit details
    Browse the repository at this point in the history

  37. feat: rudimentary setting and getting configuration status 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    249fa5e View commit details
    Browse the repository at this point in the history

  38. refactor getvalue 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    a3402a8 View commit details
    Browse the repository at this point in the history

  39. fix: don't load the .env file to the environment 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    21a46d4 View commit details
    Browse the repository at this point in the history

  40. minor: change config pre-commit hook to exit with 1 if files edited 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    a22f51a View commit details
    Browse the repository at this point in the history

  41. fix precommit requirements 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    1c7dd19 View commit details
    Browse the repository at this point in the history

  42. config: allow emojis to be set (needs more work) 

    Signed-off-by: onerandomusername <genericusername414+git@gmail.com>
    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    dd0e287 View commit details
    Browse the repository at this point in the history

  43. nit: fix typo in variable name

    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    76ff3cd View commit details
    Browse the repository at this point in the history

  44. nit: typing, formatting, and comment changes

    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    9409421 View commit details
    Browse the repository at this point in the history

  45. fix: make errors more helpful if config dependencies are not installed

    @onerandomusername
    onerandomusername committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    627e650 View commit details
    Browse the repository at this point in the history

Commits on Nov 7, 2021

  1. Merge branch 'main' into feat/improve-configuration

    @onerandomusername
    onerandomusername committed Nov 7, 2021
    Configuration menu
    Copy the full SHA
    4b8d288 View commit details
    Browse the repository at this point in the history

  2. scripts: show diff when exporting config edits files

    @onerandomusername
    onerandomusername committed Nov 7, 2021
    Configuration menu
    Copy the full SHA
    94b14e0 View commit details
    Browse the repository at this point in the history

  3. fix: no more weird new lines

    @onerandomusername
    onerandomusername committed Nov 7, 2021
    Configuration menu
    Copy the full SHA
    4117dcc View commit details
    Browse the repository at this point in the history

Commits on Nov 15, 2021

  1. Merge branch 'main' into feat/improve-configuration

    @onerandomusername
    onerandomusername committed Nov 15, 2021
    Configuration menu
    Copy the full SHA
    7743899 View commit details
    Browse the repository at this point in the history

  2. minor(config): touch up the visual UI

    @onerandomusername
    onerandomusername committed Nov 15, 2021
    Configuration menu
    Copy the full SHA
    003177e View commit details
    Browse the repository at this point in the history

  3. fix: update prefix when configured prefix is updated

    @onerandomusername
    onerandomusername committed Nov 15, 2021
    Configuration menu
    Copy the full SHA
    b4e0b7d View commit details
    Browse the repository at this point in the history

  4. minor fixes to export config script

    @onerandomusername
    onerandomusername committed Nov 15, 2021
    Configuration menu
    Copy the full SHA
    d644a96 View commit details
    Browse the repository at this point in the history

  5. nit: use dictionary instead of dit to not look like a typo

    @onerandomusername
    onerandomusername committed Nov 15, 2021
    Configuration menu
    Copy the full SHA
    52a339d View commit details
    Browse the repository at this point in the history

  6. config: rename .env.template to template.env

    @onerandomusername
    onerandomusername committed Nov 15, 2021
    Configuration menu
    Copy the full SHA
    0937a13 View commit details
    Browse the repository at this point in the history

  7. config(docs): document configuration loading priority

    @onerandomusername
    onerandomusername committed Nov 15, 2021
    Configuration menu
    Copy the full SHA
    3c60d4c View commit details
    Browse the repository at this point in the history

  8. minor: touch up comments and flatten branches

    @onerandomusername
    onerandomusername committed Nov 15, 2021
    Configuration menu
    Copy the full SHA
    5ad572d View commit details
    Browse the repository at this point in the history

Commits on Nov 16, 2021

  1. fix: use commands.run_converters() in the configuration manager 

    ensures that we use the cache and special conversion methods to convert variables
this means that all of the varaible conversion is handled by discord.py
all conversion errors are propagated outwards and caught by our error handler
    @onerandomusername
    onerandomusername committed Nov 16, 2021
    Configuration menu
    Copy the full SHA
    8bc7c27 View commit details
    Browse the repository at this point in the history

  2. config-manager: add set-default command to reset to default

    @onerandomusername
    onerandomusername committed Nov 16, 2021
    Configuration menu
    Copy the full SHA
    263ad6b View commit details
    Browse the repository at this point in the history

  3. add set default command and offload setting config to a helper method

    @onerandomusername
    onerandomusername committed Nov 16, 2021
    Configuration menu
    Copy the full SHA
    0a596b4 View commit details
    Browse the repository at this point in the history

  4. fix: add missing character to filter, fix env loading

    @onerandomusername
    onerandomusername committed Nov 16, 2021
    Configuration menu
    Copy the full SHA
    81c08da View commit details
    Browse the repository at this point in the history

  5. fix: use metadata provided converter to convert objects

    @onerandomusername
    onerandomusername committed Nov 16, 2021
    Configuration menu
    Copy the full SHA
    196d2a8 View commit details
    Browse the repository at this point in the history

  6. nits: standardize naming and make class names more self explanatory 

    cfg, config, configuration... why so many similarily named classes???
    @onerandomusername
    onerandomusername committed Nov 16, 2021
    Configuration menu
    Copy the full SHA
    ebcaf94 View commit details
    Browse the repository at this point in the history

  7. config: make dotenv optional and better errors when missing variables

    @onerandomusername
    onerandomusername committed Nov 16, 2021
    Configuration menu
    Copy the full SHA
    2fd074c View commit details
    Browse the repository at this point in the history

  8. config: add logging and stabilize configuration directory

    @onerandomusername
    onerandomusername committed Nov 16, 2021
    Configuration menu
    Copy the full SHA
    fa02b34 View commit details
    Browse the repository at this point in the history

  9. minor: use functools cache to cache the config

    @onerandomusername
    onerandomusername committed Nov 16, 2021
    Configuration menu
    Copy the full SHA
    3d1986d View commit details
    Browse the repository at this point in the history

  10. use factories to generate mutable classes and add more logging 

    tests: mark xfail the test for default config caching
    @onerandomusername
    onerandomusername committed Nov 16, 2021
    Configuration menu
    Copy the full SHA
    ff04a5a View commit details
    Browse the repository at this point in the history

  11. fix: use config.user to return configured prefix

    @onerandomusername
    onerandomusername committed Nov 16, 2021
    Configuration menu
    Copy the full SHA
    258fa1b View commit details
    Browse the repository at this point in the history

  12. chore: export the extended description after the normal description t… 

    …o app.json
    @onerandomusername
    onerandomusername committed Nov 16, 2021
    Configuration menu
    Copy the full SHA
    bfe641c View commit details
    Browse the repository at this point in the history

Commits on Nov 21, 2021

  1. chore: remove operator.attrgetter and implement recursive setters and… 

    … getters
    @onerandomusername
    onerandomusername committed Nov 21, 2021
    Configuration menu
    Copy the full SHA
    fe53cf7 View commit details
    Browse the repository at this point in the history

  2. chore: make the error spam less spammy for missing yaml file and depe… 

    …ndency
    @onerandomusername
    onerandomusername committed Nov 21, 2021
    Configuration menu
    Copy the full SHA
    4e7fb22 View commit details
    Browse the repository at this point in the history

  3. Merge branch 'main' into feat/improve-configuration

    @onerandomusername
    onerandomusername committed Nov 21, 2021
    Configuration menu
    Copy the full SHA
    dd2f16c View commit details
    Browse the repository at this point in the history

  4. tests: fix environment not being replaced soon enough

    @onerandomusername
    onerandomusername committed Nov 21, 2021
    Configuration menu
    Copy the full SHA
    03fd57f View commit details
    Browse the repository at this point in the history

Commits on Nov 25, 2021

  1. fix: use type() instead of __class__

    @onerandomusername
    onerandomusername committed Nov 25, 2021
    Configuration menu
    Copy the full SHA
    f0ec067 View commit details
    Browse the repository at this point in the history

Commits on Jan 2, 2022

  1. chore: address reviews 

    restructuring, using 'startswith', other updates
    @onerandomusername
    onerandomusername committed Jan 2, 2022
    Configuration menu
    Copy the full SHA
    09a33d4 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'main' into feat/improve-configuration

    @onerandomusername
    onerandomusername committed Jan 2, 2022
    Configuration menu
    Copy the full SHA
    8d18ccc View commit details
    Browse the repository at this point in the history

Commits on Apr 21, 2022

  1. remove dependency on environs

    @onerandomusername
    onerandomusername committed Apr 21, 2022
    Configuration menu
    Copy the full SHA
    4087767 View commit details
    Browse the repository at this point in the history

  2. fix: pin discord.py

    @onerandomusername
    onerandomusername committed Apr 21, 2022
    Configuration menu
    Copy the full SHA
    5911ff8 View commit details
    Browse the repository at this point in the history

  3. update black

    @onerandomusername
    onerandomusername committed Apr 21, 2022
    Configuration menu
    Copy the full SHA
    768277e View commit details
    Browse the repository at this point in the history

  4. update precommit hooks

    @onerandomusername
    onerandomusername committed Apr 21, 2022
    Configuration menu
    Copy the full SHA
    62e82fa View commit details
    Browse the repository at this point in the history

  5. add configuration system to docs

    @onerandomusername
    onerandomusername committed Apr 21, 2022
    Configuration menu
    Copy the full SHA
    4644d76 View commit details
    Browse the repository at this point in the history