-
Notifications
You must be signed in to change notification settings - Fork 1
Change ensure_connection to decorator #10
Copy link
Copy link
Open
Labels
good first issueGood for newcomersGood for newcomerskind/I&CCode, Infrastructure, Tests, everything that is not feature related, but more code relatedCode, Infrastructure, Tests, everything that is not feature related, but more code relatedsize/MT-shit size for tasks that take less than a weekT-shit size for tasks that take less than a weektype/enhancementNew feature or improvement that doesn't add value to final userNew feature or improvement that doesn't add value to final user
Description
Metadata
Metadata
Assignees
Labels
good first issueGood for newcomersGood for newcomerskind/I&CCode, Infrastructure, Tests, everything that is not feature related, but more code relatedCode, Infrastructure, Tests, everything that is not feature related, but more code relatedsize/MT-shit size for tasks that take less than a weekT-shit size for tasks that take less than a weektype/enhancementNew feature or improvement that doesn't add value to final userNew feature or improvement that doesn't add value to final user
Right now ensure_connection is a context manager. It's a pain in the ass to put it every single time and indent the code on it. This ticket is to transform the function into a decorator.