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
Retire Faucet #2848
Retire Faucet #2848
Conversation
all_characters = tuple(characters, ) | ||
all_configurations = tuple(configurations, ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are we no longer including blockchain_only_characters
(i.e. only StakeHolder
character) and the associated blockchain_only_configurations
(i.e. onlyStakeHolderConfiguration
)?
I know we are going to get rid of it eventually - but just checking if we wanted to keep it for now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since it's EOL I preemptively removed it here, albeit eagerly. I think the remaining tuple was empty so it was awkward to leave it dangling.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎸
RIP Felix 💀 (in this codebase).
Type of PR:
Removal
Required reviews:
1
What this does:
Felix
nucypher felix
CLIchaotic.py
Why it's needed:
Reduces the scope of the nucypher codebase.
Notes for reviewers:
We can push a git tag marking the version to use for future support. Ultimately this faucet mini-application is best served in it's own repository.