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.

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

increment_auto_off.py is difficult to sustain #971

Open
SteveMicroNova opened this issue Oct 10, 2024 · 0 comments
Open

increment_auto_off.py is difficult to sustain #971

SteveMicroNova opened this issue Oct 10, 2024 · 0 comments
Labels
tech-debt Pays down tech debt

Comments

@SteveMicroNova
Copy link
Contributor

          if this is a copy of any section of the code, you should probably create a function out of it and call it from both places

Originally posted by @rtertiaer in #828 (comment)

Ryan is right here, pay down the tech debt of copying endpoints line for line; make them importable from /usr/local/bin/increment_auto_off.py

I attempted to do this using a symbolic link and relative import, ran into an issue where increment_auto_off.py isn't inside the venv and so doesn't have access to all required dependencies and wasn't able to make said dependencies install globally via configure.py either

@SteveMicroNova SteveMicroNova added the tech-debt Pays down tech debt label Oct 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tech-debt Pays down tech debt
Projects
None yet
Development

No branches or pull requests

1 participant