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

Fix help text for exaslct #253

Closed
tomuben opened this issue Jan 2, 2025 · 0 comments · Fixed by #262
Closed

Fix help text for exaslct #253

tomuben opened this issue Jan 2, 2025 · 0 comments · Fixed by #262
Assignees
Labels
refactoring Code improvement without behavior change

Comments

@tomuben
Copy link
Collaborator

tomuben commented Jan 2, 2025

The help text (when running exaslct --help) shows:

Usage: exaslct [OPTIONS] COMMAND [ARGS]...

  ITDE - Integration Test Docker Environment

  Create and control a docker based exasol database test setup.

  Examples:

      Check the health of the execution environment:

          $ itde health

      Spawn an ITDE test environment:

          $ itde spawn-test-environment --environment-name test \
          --database-port-forward 8563 --bucketfs-port-forward 2580 \
          --docker-db-image-version 7.1.9 --db-mem-size 4GB

Options:
  --help  Show this message and exit.

Commands:
  build                         Builds a script-languages-container.
...

=> The commands are correct, but the heading is used from itde.
The reason is that we use the same cli() routine from itde, see:

from exasol_integration_test_docker_environment.cli.cli import cli # type: ignore

We need to implement a separate routine with it's own help text.

@tomuben tomuben added the documentation User guides, tutorials, specifications label Jan 2, 2025
@tomuben tomuben added refactoring Code improvement without behavior change and removed documentation User guides, tutorials, specifications labels Jan 28, 2025
tomuben added a commit that referenced this issue Jan 28, 2025
@tomuben tomuben self-assigned this Jan 29, 2025
tomuben added a commit that referenced this issue Jan 29, 2025
fixes #253 

Co-authored-by: Torsten Kilias <tkilias@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring Code improvement without behavior change
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant