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

Adding format_down option to path_exists and run_watch #17

Merged
merged 1 commit into from
Apr 2, 2015
Merged

Adding format_down option to path_exists and run_watch #17

merged 1 commit into from
Apr 2, 2015

Conversation

chrko
Copy link
Contributor

@chrko chrko commented Apr 2, 2015

According to #1 the option format_down (to hide the output) is missing on path_exists. This also add the option to run_watch.
Do not change behavior without editing config, so no user will claim the new optional behaviour!

@@ -259,6 +259,8 @@ to the config section.
Expands the given path to a pidfile and checks if the process ID found inside
is valid (that is, if the process is running). You can use this to check if
a specific application, such as a VPN client or your DHCP client is running.
It also exists an option "format_down". You can hide the output with
Copy link
Member

Choose a reason for hiding this comment

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

“It also exists” is a german idiom that doesn’t translate directly to english. Use “There also is” instead. Also, please replace the quotes with +, i.e. +format_down=""+ to be consistent with the rest of this manpage.

@stapelberg
Copy link
Member

Please squash your commits into one commit so that we have a clean history.

Also, please add “fixes #1” to the commit description so that the issue will automatically be closed.

@chrko
Copy link
Contributor Author

chrko commented Apr 2, 2015

Squashed it.

stapelberg added a commit that referenced this pull request Apr 2, 2015
Adding format_down option to path_exists and run_watch
@stapelberg stapelberg merged commit d00a0e0 into i3:master Apr 2, 2015
@chrko chrko deleted the format_down branch April 2, 2015 20:43
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