Skip to content

Commit

Permalink
Merge pull request #355 from zapta/develop
Browse files Browse the repository at this point in the history
Added the alias -h to the existing --help flag.
  • Loading branch information
Obijuan authored Feb 26, 2024
2 parents 789efb7 + 1dcf9d2 commit 9b348ad
Show file tree
Hide file tree
Showing 20 changed files with 41 additions and 19 deletions.
2 changes: 1 addition & 1 deletion apio/__main__.py
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,7 @@ def get_command(self, ctx, cmd_name: string):
# -- This function is executed when apio is executed without
# -- any parameter. The help is shown
# ------------------------------------------------------------------
@click.command(cls=ApioCLI, invoke_without_command=True)
@click.command(cls=ApioCLI, invoke_without_command=True, context_settings=util.context_settings())
@click.pass_context
@click.version_option()
def cli(ctx):
Expand Down
3 changes: 2 additions & 1 deletion apio/commands/boards.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,10 @@
import click

from apio.resources import Resources
from apio import util


@click.command("boards")
@click.command("boards", context_settings=util.context_settings())
@click.pass_context
@click.option(
"-l", "--list", is_flag=True, help="List all supported FPGA boards."
Expand Down
3 changes: 2 additions & 1 deletion apio/commands/build.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,13 +9,14 @@
import click

from apio.managers.scons import SCons
from apio import util


# R0913: Too many arguments (6/5)
# pylint: disable=R0913
# pylint: disable=W0622
# pylint: disable=R0801
@click.command("build")
@click.command("build", context_settings=util.context_settings())
@click.pass_context
@click.option(
"-b", "--board", type=str, metavar="board", help="Set the board."
Expand Down
3 changes: 2 additions & 1 deletion apio/commands/clean.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,10 @@
import click

from apio.managers.scons import SCons
from apio import util


@click.command("clean")
@click.command("clean", context_settings=util.context_settings())
@click.pass_context
@click.option(
"-p",
Expand Down
3 changes: 2 additions & 1 deletion apio/commands/config.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,10 +8,11 @@
import click

from apio.profile import Profile
from apio import util


# pylint: disable=W0622
@click.command("config")
@click.command("config", context_settings=util.context_settings())
@click.pass_context
@click.option(
"-l", "--list", is_flag=True, help="List all configuration parameters."
Expand Down
3 changes: 2 additions & 1 deletion apio/commands/drivers.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,10 @@
import click

from apio.managers.drivers import Drivers
from apio import util


@click.command("drivers")
@click.command("drivers", context_settings=util.context_settings())
@click.pass_context
@click.option("--ftdi-enable", is_flag=True, help="Enable FTDI drivers.")
@click.option("--ftdi-disable", is_flag=True, help="Disable FTDI drivers.")
Expand Down
3 changes: 2 additions & 1 deletion apio/commands/examples.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,12 +8,13 @@
import click

from apio.managers.examples import Examples
from apio import util


# R0913: Too many arguments (6/5)
# pylint: disable=R0913
# pylint: disable=W0622
@click.command("examples")
@click.command("examples", context_settings=util.context_settings())
@click.pass_context
@click.option(
"-l", "--list", is_flag=True, help="List all available examples."
Expand Down
3 changes: 2 additions & 1 deletion apio/commands/init.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,11 +8,12 @@
import click

from apio.managers.project import Project
from apio import util


# R0913: Too many arguments (6/5)
# pylint: disable=R0913
@click.command("init")
@click.command("init", context_settings=util.context_settings())
@click.pass_context
@click.option(
"-b",
Expand Down
3 changes: 2 additions & 1 deletion apio/commands/install.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@

from apio.managers.installer import Installer
from apio.resources import Resources
from apio import util

# R0801: Similar lines in 2 files
# pylint: disable=R0801
Expand All @@ -30,7 +31,7 @@
# R0913: Too many arguments (6/5)
# pylint: disable=R0913
# pylint: disable=W0622
@click.command("install")
@click.command("install", context_settings=util.context_settings())
@click.pass_context
@click.argument("packages", nargs=-1)
@click.option("-a", "--all", is_flag=True, help="Install all packages.")
Expand Down
3 changes: 2 additions & 1 deletion apio/commands/lint.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,12 +8,13 @@
import click

from apio.managers.scons import SCons
from apio import util


# R0913: Too many arguments (6/5)
# pylint: disable=R0913
# pylint: disable=W0622
@click.command("lint")
@click.command("lint", context_settings=util.context_settings())
@click.pass_context
@click.option(
"-a",
Expand Down
2 changes: 1 addition & 1 deletion apio/commands/raw.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
from apio import util


@click.command("raw")
@click.command("raw", context_settings=util.context_settings())
@click.pass_context
@click.argument("cmd")
def cli(ctx, cmd):
Expand Down
3 changes: 2 additions & 1 deletion apio/commands/sim.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,9 +7,10 @@

import click
from apio.managers.scons import SCons
from apio import util


@click.command("sim")
@click.command("sim", context_settings=util.context_settings())
@click.pass_context
@click.option(
"-p",
Expand Down
3 changes: 2 additions & 1 deletion apio/commands/system.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,11 +7,12 @@

import click

from apio import util
from apio.util import get_systype
from apio.managers.system import System


@click.command("system")
@click.command("system", context_settings=util.context_settings())
@click.pass_context
@click.option(
"--lsftdi", is_flag=True, help="List all connected FTDI devices."
Expand Down
3 changes: 2 additions & 1 deletion apio/commands/test.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,9 +7,10 @@

import click
from apio.managers.scons import SCons
from apio import util


@click.command("test")
@click.command("test", context_settings=util.context_settings())
@click.pass_context
@click.option(
"-p",
Expand Down
3 changes: 2 additions & 1 deletion apio/commands/time.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,12 +8,13 @@
import click

from apio.managers.scons import SCons
from apio import util


# R0913: Too many arguments (6/5)
# pylint: disable=R0913
# pylint: disable=W0622
@click.command("time")
@click.command("time", context_settings=util.context_settings())
@click.pass_context
@click.option(
"-b", "--board", type=str, metavar="board", help="Set the board."
Expand Down
3 changes: 2 additions & 1 deletion apio/commands/uninstall.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@
from apio.managers.installer import Installer
from apio.resources import Resources
from apio.profile import Profile
from apio import util

platforms = [
"linux_x86_64",
Expand All @@ -23,7 +24,7 @@


# pylint: disable=W0622
@click.command("uninstall")
@click.command("uninstall", context_settings=util.context_settings())
@click.pass_context
@click.argument("packages", nargs=-1)
@click.option("-a", "--all", is_flag=True, help="Uninstall all packages.")
Expand Down
3 changes: 2 additions & 1 deletion apio/commands/upgrade.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,9 +10,10 @@

from packaging import version
from apio.util import get_pypi_latest_version
from apio import util


@click.command("upgrade")
@click.command("upgrade", context_settings=util.context_settings())
@click.pass_context
def cli(ctx):
"""Check the latest Apio version."""
Expand Down
3 changes: 2 additions & 1 deletion apio/commands/upload.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,11 +9,12 @@

from apio.managers.scons import SCons
from apio.managers.drivers import Drivers
from apio import util


# R0913: Too many arguments (6/5)
# pylint: disable=R0913
@click.command("upload")
@click.command("upload", context_settings=util.context_settings())
@click.pass_context
@click.option(
"-b", "--board", type=str, metavar="board", help="Set the board."
Expand Down
3 changes: 2 additions & 1 deletion apio/commands/verify.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,10 @@
import click

from apio.managers.scons import SCons
from apio import util


@click.command("verify")
@click.command("verify", context_settings=util.context_settings())
@click.pass_context
@click.option(
"-p",
Expand Down
5 changes: 5 additions & 0 deletions apio/util.py
Original file line number Diff line number Diff line change
Expand Up @@ -807,3 +807,8 @@ def get_python_version():
"""Return a string with the python version"""

return f"{sys.version_info[0]}.{sys.version_info[1]}"

def context_settings():
"""Return a common Click command settings that adds the alias -h to --help"""
# Per https://click.palletsprojects.com/en/8.1.x/documentation/#help-parameter-customization
return dict(help_option_names=['-h', '--help'])

0 comments on commit 9b348ad

Please sign in to comment.