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

Don't check size of PGP fingerprint if it's not set #110

Merged
merged 2 commits into from
Sep 23, 2024

Conversation

thewhaleking
Copy link
Contributor

Fixes #109

@thewhaleking thewhaleking merged commit 25586af into main Sep 23, 2024
3 checks passed
@thewhaleking thewhaleking deleted the fix/thewhaleking/pgp-required branch September 23, 2024 16:10
ibraheem-opentensor pushed a commit that referenced this pull request Sep 23, 2024
* Give user the UID immediately after root registration.

* Only check size of PGP fingerprint if it's actually set.
ibraheem-opentensor added a commit that referenced this pull request Sep 23, 2024
* Adds support for passing comma sep values

* Adds func to salt

* Seventh set of Typer docstrings

* Table fixes / E2E Test  - Senate command fix (#100)

* Init commit

* Fixes table

* Working

* Adds back Hotkey

* Adds separation in overview

* WIP: Senate

* Finishes senate e2e

* Fixes tests

* Update help.

* Ruff

* Give user their UID immediately after root/sn registration. (#108)

* Give user the UID immediately after root registration.

* Subnet register

* Don't check size of PGP fingerprint if it's not set (#110)

* Give user the UID immediately after root registration.

* Only check size of PGP fingerprint if it's actually set.

* Enhancement: max-stake and table tweak (#105)

* Enhances max_stake and column change in list-delegates

* Ruff

* Ruff

---------

Co-authored-by: Benjamin Himes <benhimes@opentensor.dev>

* Propagates list supp, exists in-case of no subnet in sudo set, fixes staking multiple

* Ruff

* Adds support for both hotkeys and ss58s

* Updated help docstring for stake_remove to reflect new list style

---------

Co-authored-by: Watchmaker <rajk@opentensor.dev>
Co-authored-by: Benjamin Himes <benhimes@opentensor.dev>
Co-authored-by: Benjamin Himes <37844818+thewhaleking@users.noreply.github.com>
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.

PGP Fingerprint is required
2 participants