Skip to content

Conversation

@drq883
Copy link

@drq883 drq883 commented Jul 1, 2023

This make it easier

@drq883
Copy link
Author

drq883 commented Jul 1, 2023

Another question, why rename it from awake?

@Post2FixO
Copy link
Contributor

Hi @drq883. Just saw this while going over my repositories. It was a fun app to write. I'll try to merge your changes. There seem to be conflicts so I'll try go over them.

About the name, short for Tesrminal Wake and play on Terminal patient with insomnia.. Don't think anyone would give it enough thought to get it. I wanted a unique name but a different one would probably be better.

Hope you enjoyed this script. I was aiming to establish core functionalities that are not available in more popular apps, such as sleep after, wake locked mode, sleep duration, external screen lid control logic (mMac stays unlocked on lid close when ext monitor is connected), etc.

Thanks for youe contribution!

Copy link
Contributor

@Post2FixO Post2FixO left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think I was trying to avoid the need for providing sudo creds so the app can run in the background and these parameters were editable without providing creds (or we could do a one time permission grant). But if this made it easier for you, I'll merge it. Thanks for your contribution!

I renamed to TerminalWake. At least that one less neurological step to perform :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants