-
Notifications
You must be signed in to change notification settings - Fork 86
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
Adjust issue template and PR template to fit OpenBao #5
Adjust issue template and PR template to fit OpenBao #5
Conversation
Personally I like bao. It looks like opentofu uses tofu, so it would make sense to follow their lead |
Agree @jbutlerdev. I don't believe there are any popular |
Make sure all commits use |
Signed-off-by: Andrew Savchyn <dev@scorpil.com>
359f48e
to
0b1971e
Compare
- type: textarea | ||
id: security | ||
attributes: | ||
label: Security Implications |
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.
I don't think its worth highlighting everything in the RFC design, but perhaps adding user experience here would be a good catch-all bucket: e.g., developer experience integrating with this proposed change, operator experience trying to manage/configure it, ...
Just to make sure it gets thought about before dropping a (complex but useful) feature in...
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.
Fair enough, added ✅
…ction in RFC template Signed-off-by: Andrew Savchyn <dev@scorpil.com>
eb8f9bb
to
8ae7e24
Compare
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.
Looks good to me!
Solves #3
Removing plugin submission template for now.
Added RFC template based on OpenTofu's with slight modifications.
Labels are configured based on #4