Skip to content

Commit

Permalink
Improve cli help messages (#185)
Browse files Browse the repository at this point in the history
  • Loading branch information
picklelo authored Dec 27, 2022
1 parent a93b142 commit 2be2074
Show file tree
Hide file tree
Showing 4 changed files with 17 additions and 33 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -50,5 +50,5 @@ jobs:
- run: poetry run pytest tests
- run: poetry run pyright pynecone tests
- run: poetry run pydocstyle pynecone tests
- run: poetry run darglint pynecone tests
- run: find pynecone tests -name "*.py" -not -path pynecone/pc.py | xargs poetry run darglint
- run: poetry run black --check pynecone tests
8 changes: 4 additions & 4 deletions pynecone/app.py
Original file line number Diff line number Diff line change
Expand Up @@ -85,10 +85,10 @@ def __call__(self) -> FastAPI:
def add_default_endpoints(self):
"""Add the default endpoints."""
# To test the server.
self.api.get(str(constants.Endpoint.PING))(_ping)
self.api.get(str(constants.Endpoint.PING))(ping)

# To make state changes.
self.api.websocket(str(constants.Endpoint.EVENT))(_event(app=self))
self.api.websocket(str(constants.Endpoint.EVENT))(event(app=self))

def add_cors(self):
"""Add CORS middleware to the app."""
Expand Down Expand Up @@ -290,7 +290,7 @@ def set_state(self, token: str, state: State):
self.state_manager.set_state(token, state)


async def _ping() -> str:
async def ping() -> str:
"""Test API endpoint.
Returns:
Expand All @@ -299,7 +299,7 @@ async def _ping() -> str:
return "pong"


def _event(app: App):
def event(app: App):
"""Websocket endpoint for events.
Args:
Expand Down
38 changes: 12 additions & 26 deletions pynecone/pc.py
Original file line number Diff line number Diff line change
Expand Up @@ -19,11 +19,7 @@ def version():

@cli.command()
def init():
"""Initialize a new Pynecone app.
Raises:
Exit: If the app directory is invalid.
"""
"""Initialize a new Pynecone app in the current directory."""
app_name = utils.get_default_app_name()

# Make sure they don't name the app "pynecone".
Expand All @@ -49,22 +45,16 @@ def init():

@cli.command()
def run(
env: constants.Env = constants.Env.DEV,
frontend: bool = True,
backend: bool = True,
loglevel: constants.LogLevel = constants.LogLevel.ERROR,
env: constants.Env = typer.Option(
constants.Env.DEV, help="The environment to run the app in."
),
frontend: bool = typer.Option(True, help="Whether to run the frontend."),
backend: bool = typer.Option(True, help="Whether to run the backend."),
loglevel: constants.LogLevel = typer.Option(
constants.LogLevel.ERROR, help="The log level to use."
),
):
"""Run the app.
Args:
env: The environment to run the app in.
frontend: Whether to run the frontend.
backend: Whether to run the backend.
loglevel: The log level to use.
Raises:
Exit: If the app is not initialized.
"""
"""Run the app in the current directory."""
# Check that the app is initialized.
if frontend and not utils.is_initialized():
utils.console.print(
Expand Down Expand Up @@ -99,12 +89,8 @@ def run(


@cli.command()
def deploy(dry_run: bool = False):
"""Deploy the app to the hosting service.
Args:
dry_run: Whether to run a dry run.
"""
def deploy(dry_run: bool = typer.Option(False, help="Whether to run a dry run.")):
"""Deploy the app to the Pynecone hosting service."""
# Get the app config.
config = utils.get_config()
config.api_url = utils.get_production_backend_url()
Expand Down
2 changes: 0 additions & 2 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -52,5 +52,3 @@ pc = "pynecone.pc:main"
[build-system]
requires = ["poetry-core>=1.0.0"]
build-backend = "poetry.core.masonry.api"

[tool.pyright]

0 comments on commit 2be2074

Please sign in to comment.