Skip to content

Removed root user check, just use sudo#1

Open
drq883 wants to merge 1 commit intoPost2Fix:mainfrom
drq883:main
Open

Removed root user check, just use sudo#1
drq883 wants to merge 1 commit intoPost2Fix:mainfrom
drq883:main

Conversation

@drq883
Copy link
Copy Markdown

@drq883 drq883 commented Jul 1, 2023

This make it easier

@drq883
Copy link
Copy Markdown
Author

drq883 commented Jul 1, 2023

Another question, why rename it from awake?

@Post2FixO
Copy link
Copy Markdown
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
Copy Markdown
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