Skip to content

Commit

Permalink
Argcomplete support for client + log tweak
Browse files Browse the repository at this point in the history
  • Loading branch information
zwimer committed Oct 28, 2024
1 parent bf57c84 commit ec52873
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 2 deletions.
1 change: 1 addition & 0 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ dependencies = [
"zstandard[cffi]",
"pycryptodomex",
"cryptography",
"argcomplete",
"requests",
"tqdm",
# Server
Expand Down
2 changes: 1 addition & 1 deletion rpipe/__init__.py
Original file line number Diff line number Diff line change
@@ -1 +1 @@
__version__: str = "9.3.2" # Must be "<major>.<minor>.<patch>", all numbers
__version__: str = "9.4.0" # Must be "<major>.<minor>.<patch>", all numbers
3 changes: 3 additions & 0 deletions rpipe/client/cli.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@
from pathlib import Path
import argparse

import argcomplete

from .. import __version__ # Extract version without importing shared


Expand Down Expand Up @@ -166,4 +168,5 @@ def cli() -> None:
log_lvl_p.add_argument("level", default=None, nargs="?", help="The log level for the server to use")
admin.add_parser("lock", help="Lock the channel")
admin.add_parser("unlock", help="Unlock the channel")
argcomplete.autocomplete(parser) # Tab completion
_cli(parser, parser.parse_args())
2 changes: 1 addition & 1 deletion rpipe/server/server/server.py
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ def __init__(self, debug: bool, state_file: Path | None) -> None:
self._log.info("Skipping initialization until reload")
return
if not debug: # Don't do this for debug, it will be overridden
self._log.warning("Installing signal handlers to ensure graceful shutdown")
self._log.info("Installing signal handlers to ensure graceful shutdown")
for i in self.SIGNALS:
signal.signal(i, _ctrlc)
# Load state file as needed
Expand Down

0 comments on commit ec52873

Please sign in to comment.