-
Notifications
You must be signed in to change notification settings - Fork 99
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
chore: issue templates #716
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: dextrot <[email protected]>
Signed-off-by: dextrot <[email protected]>
added the issue templates here @FeynmanZhou @shizhMSFT |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #716 +/- ##
=======================================
Coverage 75.92% 75.92%
=======================================
Files 59 59
Lines 5873 5873
=======================================
Hits 4459 4459
Misses 1037 1037
Partials 377 377 ☔ View full report in Codecov by Sentry. |
Signed-off-by: SKM <[email protected]>
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.
@dextrot Issue templates for oras-go
should not be directly copied from the oras
repo since the SDK is very different from a CLI.
Consider adapting issue templates from notation-go, which is also a golang library.
required: true | ||
attributes: | ||
label: What happened in your environment? | ||
description: "Please also attach logs here using `--debug` flag." |
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 oras-go
is a library, there is no --debug
flag.
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.
@shizhMSFT can we put the description like : description: "Please also attach logs here ."
?
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.
can we put the description like : description: "Please also attach logs here ." ?
This sounds good to me.
label: What is the version of your ORAS CLI? | ||
description: "You can use the command `oras version` to get it." |
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.
Again, it is oras-go
library, not a CLI. There is no oras version
command for it.
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.
This also applies to feature-request.yaml
.
Co-authored-by: Feynman Zhou <[email protected]> Signed-off-by: SKM <[email protected]>
Co-authored-by: Feynman Zhou <[email protected]> Signed-off-by: SKM <[email protected]>
Co-authored-by: Feynman Zhou <[email protected]> Signed-off-by: SKM <[email protected]>
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.
Thanks @dextrot . Please do not replicate the ORAS issue template here as they have different functionalities. PTAL at my comments above
Signed-off-by: SKM <[email protected]>
Signed-off-by: SKM <[email protected]>
Signed-off-by: SKM <[email protected]>
@dextrot Thanks for the contribution! Could you address the remaining comments and rebase? |
This PR is stale because it has been open for 45 days with no activity. Remove the stale label or comment to prevent it from being closed in 30 days. |
This PR fixes #641
Bug report
,feature request
and the config file for the templates.