Skip to content
This repository has been archived by the owner on Jan 1, 2023. It is now read-only.

Restructured module imports & exports #72

Merged
merged 1 commit into from
Sep 30, 2022
Merged

Restructured module imports & exports #72

merged 1 commit into from
Sep 30, 2022

Conversation

fdnt7
Copy link
Owner

@fdnt7 fdnt7 commented Sep 30, 2022

* __init__ files will now only be for exporting vars
| / separated everything that was originally in there to a separate file
| * every imports will now only use top level modules with the exported vars
* Reordered every imports by relevancy relative to the current file
+ LyraConfig class
| * encapsulates config data from config.yml
| + as an injected dependency
| ! fixed misc module tracking commands of the wrong guild
- music file
* musicutils -> music
- removed a lot of in-function imports
* all errors will now have ~Error as a suffix
/ seperated BaseLyraSignal to be bases of ChannelMoved and RequestedToSpeak

`*` `__init__` files will now only be for exporting vars
 | `/` separated everything that was originally in there to a separate file
 | `*` every imports will now only use top level modules with the exported vars
`*` Reordered every imports by relevancy relative to the current file
`+` `LyraConfig` class
 | `*` encapsulates config data from `config.yml`
 | `+` as an injected dependency
 | `!` fixed `misc` module tracking commands of the wrong guild
`-` `music` file
`*` `musicutils` -> `music`
`-` removed a lot of in-function imports
`*` all errors will now have `~Error` as a suffix
`/` seperated `BaseLyraSignal` to be bases of `ChannelMoved` and `RequestedToSpeak`
@fdnt7 fdnt7 self-assigned this Sep 30, 2022
@fdnt7 fdnt7 added breaking This change hurts backwards compatibility bug Something isn't working enhancement New feature or request optimization Optimizations to the code labels Sep 30, 2022
@fdnt7 fdnt7 merged commit e1aab7c into main Sep 30, 2022
@fdnt7 fdnt7 deleted the modules-restructure branch September 30, 2022 15:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
breaking This change hurts backwards compatibility bug Something isn't working enhancement New feature or request optimization Optimizations to the code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant