Skip to content

E2E tests + fixes#92

Merged
ibraheem-abe merged 8 commits intomainfrom
tests/abe/adds-more-tests-and-fixes
Sep 19, 2024
Merged

E2E tests + fixes#92
ibraheem-abe merged 8 commits intomainfrom
tests/abe/adds-more-tests-and-fixes

Conversation

@ibraheem-abe
Copy link
Copy Markdown
Contributor

@ibraheem-abe ibraheem-abe commented Sep 19, 2024

  • Adds --no-use-hotkey for set_id
  • Fixes stake show prompting in-case --all used
  • Enhances prompt text for stake add/remove
  • Adds validate_coldkey_presence utility and used in stake show
  • Adds e2e for set-id, get-id, sign commands

thewhaleking
thewhaleking previously approved these changes Sep 19, 2024
Comment thread bittensor_cli/cli.py Outdated
@ibraheem-abe ibraheem-abe merged commit f71ce24 into main Sep 19, 2024
thewhaleking added a commit that referenced this pull request Sep 19, 2024
* Second set of Typer docstrings

* Third set of Typer docstrings (#94)

* Third set of Typer docstrings

* Fix into third set

* Removes newdoc.md

---------

Co-authored-by: ibraheem-opentensor <ibraheem@opentensor.dev>

* E2E tests + fixes (#92)

* Fixes help text

* Adds set and get id tests

* Adds flags for wallet_sign and improves prompt

* Adds wallet set-id, get-id, sign tests

* Fixes stake show command edge cases

* Removes accidental readme file

* Removes explicit email flag

* Decode CHK SS58 (#95)

* Fix child hotkeys by decoding the ss58

* Race condition

* Fixes stake show command edge cases

* Removes accidental readme file

* WIP

* Made call_add_proposal work

* Adds proposal list call

* Fixed dict vs class change.

---------

Co-authored-by: Watchmaker <rajk@opentensor.dev>
Co-authored-by: Benjamin Himes <37844818+thewhaleking@users.noreply.github.com>
Co-authored-by: Benjamin Himes <benhimes@opentensor.dev>
@ibraheem-abe ibraheem-abe mentioned this pull request Sep 25, 2024
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