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.

Sign up for GitHub

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

Jump to bottom

Use separate required option for @cog.input() #114

Closed
bfirsh opened this issue Jun 4, 2021 · 1 comment
Closed

Use separate required option for @cog.input() #114

bfirsh opened this issue Jun 4, 2021 · 1 comment
Projects
Priorities

Comments

@bfirsh
Copy link
Member

bfirsh commented Jun 4, 2021 •
edited

The double behavior of default is not obvious.

This might want to be optional=True instead, so that inputs are required by default. If the inputs were not required by default, then users normally would make them required. If required inputs are not marked as required, this will cause breakage.
@bfirsh bfirsh mentioned this issue Aug 27, 2021
Closed
@bfirsh bfirsh mentioned this issue Jan 13, 2022
Merged
36 tasks
@zeke zeke added this to Awaiting Triage in Priorities Mar 14, 2022
@zeke
Copy link
Member

zeke commented Mar 14, 2022

Closing, as @cog.input has been removed as of v0.1.0-alpha.

@zeke zeke closed this as completed Mar 14, 2022
Priorities automation moved this from Awaiting Triage to Done Mar 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
Priorities
Done
Milestone
No milestone
Development

No branches or pull requests
2 participants
@zeke @bfirsh