-
Notifications
You must be signed in to change notification settings - Fork 71
[CDTOOL-1299] Improve Issue Template Routing #1705
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
base: main
Are you sure you want to change the base?
Changes from all commits
6b024fc
d371684
36c0e68
719f911
704ee74
2aaa4ef
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,2 @@ | ||
| # Disable blank issues for non-maintainers. | ||
| blank_issues_enabled: false | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,34 @@ | ||
| <div align="center"> | ||
| <h3 align="center">CLI Issues</h3> | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I am very open to suggestions on how this document should be organized and formatted. Some of these examples may not be perfect - so please feel free to suggest improvements there as well. |
||
| <p align="center">Best practices for submitting an issue to the Fastly CLI repository.</p> | ||
| </div> | ||
|
|
||
| ## Issue Type: Bug | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There are two other aspects I'd like to try to consider here:
In other words, my personal preference would be to (as gently as possible) discourage customers from opening bug reports or feature requests in the public repositories at all, unless it is very clear to them that they will be comfortable with the discussion occurring in public and that the response time for public issues will be somewhat longer than for private support tickets.
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Got it - it sounds like we'll be wanting to add this verbiage to both the issue templates as well as the ISSUES.md file then. |
||
|
|
||
| Issues related to the CLI behavior not working as intended. | ||
|
|
||
| - The CLI crashes or exits with an unexpected error | ||
| - A command produces incorrect output or wrong results | ||
| - Commands or flags don't work as documented | ||
|
|
||
| **Example:** "When I run `fastly service list --json`, malformed JSON is produced." | ||
|
|
||
| ## Issue Type: Feature Request | ||
|
|
||
| Issues related to suggesting improvements to the CLI: | ||
|
|
||
| - New commands or subcommands based on existing Fastly APIs | ||
| - Improved error messages or user experience | ||
| - Adding support for a third party integration | ||
|
|
||
| **Example:** "Add a `fastly service version validate` command, which already exists in the Fastly API." | ||
|
|
||
| ## Fastly Support | ||
|
|
||
| CLI behavior specific to your environment or service / account should be routed to the Fastly support team @ support.fastly.com or support@fastly.com. | ||
|
|
||
| - A feature is missing from your account / service | ||
| - Partial content is returned that you may not have access to with your current Fastly account role | ||
| - My site is not loading after a configuration change | ||
|
|
||
| **Example:** When running `fastly service vcl snippet create`, an error is thrown that the provided VCL is not valid | ||
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 forces users to chose one of the existing issue types,
BugorFeature Request.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.
Maintainers however, will still be able to use the
Blankissue type if needed.