Skip to content

Alphabetize Configs#1380

Merged
Xithrius merged 3 commits into
python-discord:masterfrom
HassanAbouelela:alphabetize-config
Jan 26, 2021
Merged

Alphabetize Configs#1380
Xithrius merged 3 commits into
python-discord:masterfrom
HassanAbouelela:alphabetize-config

Conversation

@HassanAbouelela
Copy link
Copy Markdown
Contributor

@HassanAbouelela HassanAbouelela commented Jan 24, 2021

Closes #1312.

Reorganizes bot config files alphabetically. Certain considerations went into organizing things. constants.py mirrors default-config where possible.

While most of the file is now organized alphabetically, some exceptions were made in locations where it either wasn't possible due to references, would've made navigation more difficult, or where discreet groupings were made within the same scope.

@HassanAbouelela HassanAbouelela added t: feature New feature or request p: 3 - low Low Priority labels Jan 24, 2021
@coveralls
Copy link
Copy Markdown

coveralls commented Jan 24, 2021

Coverage Status

Coverage remained the same at 56.655% when pulling 2eb51b6 on HassanAbouelela:alphabetize-config into 134a2c6 on python-discord:master.

@Xithrius Xithrius added a: backend Related to internal functionality and utilities (error_handler, logging, security, utils and core) l: 0 - beginner s: WIP Work In Progress labels Jan 24, 2021
Mirrors the changes from config-default.yml to constants.py.
@HassanAbouelela HassanAbouelela marked this pull request as ready for review January 24, 2021 17:10
@Xithrius Xithrius removed the s: WIP Work In Progress label Jan 26, 2021
@Xithrius Xithrius merged commit 212d065 into python-discord:master Jan 26, 2021
@HassanAbouelela HassanAbouelela deleted the alphabetize-config branch January 27, 2021 07:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: backend Related to internal functionality and utilities (error_handler, logging, security, utils and core) l: 0 - beginner p: 3 - low Low Priority t: feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Alphabetize Default Config

3 participants