Skip to content

Functions reformat#52

Open
MrOinky wants to merge 6 commits intomasterfrom
functions-reformat
Open

Functions reformat#52
MrOinky wants to merge 6 commits intomasterfrom
functions-reformat

Conversation

@MrOinky
Copy link
Collaborator

@MrOinky MrOinky commented Jan 3, 2022

Splits functions.py up into the utils folder so that functions are more organised.

The current structure is like so:

-utils
  -sql
    -db.py   - File for useful functions relating to the database e.g. backup functionality.
    -sql.py  - File for functions relating to interacting with the data inside the database.
  -checks.py - File for various different types of checks, e.g. role checks.
  -time.py   - File for useful functions for working with time,
  -utils.py  - File which serves a purpose like functions.py, where generic utilities reside which are not major enough to exist in their own file.

More files can be added in the future, the current files have only been created based off of what was previously in functions.py.

Closes #50

All time related functions have been moved from functions.py to utils/time.py.
All check-related functions from functions.py now reside in utils/checks.py
SQL related tools now reside within utils/sql. These are split between sql.py and db.py, depending on their purpose.
Move generic utilities left in functions.py into utils/utils.py where they don't necessarily need their own categories.
@MrOinky MrOinky added the enhancement New feature or request label Jan 3, 2022
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.

Reformat functions.py

2 participants