-
Notifications
You must be signed in to change notification settings - Fork 439
Added github issues templates #1952
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
Changes from 18 commits
e927b0c
a0a074c
89528a7
42c06eb
b5dbb51
ea7855b
9e6656b
dfb8fc9
d5303ec
1ff166c
8cdad46
ed57b88
1787248
b19b9df
3a72cfa
c1f0d49
f2e1b02
9a3b198
3d83efb
cb971b2
8cf2e37
755a79b
64c02f8
d306aa4
5ce2fe6
5170573
978f584
21644e4
86eca89
230f958
c2d5d65
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,6 @@ | ||
blank_issues_enabled: true | ||
issue_templates: | ||
- name: Web API type definition issue | ||
description: Report inconsistencies between TypeScript's web API typings and browser behavior. | ||
labels: [lib.d.ts bug] | ||
file: web_api_type_definition_issue.yml |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,68 @@ | ||
name: "Web API type definition issue" | ||
title: "[Web API type definition issue] " | ||
labels: [lib.d.ts bug] | ||
assignees: [] | ||
description: "Report inconsistencies between TypeScript's web API typings and browser behavior." | ||
body: | ||
- type: markdown | ||
attributes: | ||
value: | | ||
## Web API type definition issue | ||
Provide details of the issue. | ||
- type: input | ||
id: issue_summary | ||
attributes: | ||
label: "Summary" | ||
description: "Brief summary of the issue" | ||
placeholder: "e.g., Mismatch in event handling..." | ||
validations: | ||
required: true | ||
- type: textarea | ||
id: expected_vs_actual | ||
attributes: | ||
label: "Expected vs. Actual Behavior" | ||
description: "Describe what you expected and what actually happened" | ||
placeholder: "Expected: ...\nActual: ..." | ||
validations: | ||
required: true | ||
- type: textarea | ||
id: steps | ||
attributes: | ||
label: "Playground link" | ||
description: "Paste the playground link" | ||
placeholder: "https://www.typescriptlang.org/play/..." | ||
validations: | ||
required: true | ||
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. Are these elements required on purpose? Is the intent that anything outside this specific format should be a blank issue? I can imagine someone reporting a bug without a playground link. 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. Yes, this is my intent because we need a playground link to help us reproduce the issue. If you think it shouldn't be required I can do that :) 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. If @saschanaz is happy, then I'm happy; I'm just double checking as I haven't read everything that happened over the course of the PR. 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'm fine with making it not required, sometimes it's about conflict with third party type definitions and that wouldn't be reproducible with playground. 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 have made it optional @jakebailey |
||
- type: checkboxes | ||
id: browser_support | ||
attributes: | ||
label: "Browser Support" | ||
description: "Ensure that the API is supported in at least two major browser engines (not two chromiums)." | ||
options: | ||
- label: "This API is supported in at least two major browser engines (not two chromiums)." | ||
validations: | ||
required: true | ||
- type: checkbox | ||
id: use_latest_typescript | ||
attributes: | ||
label: "Use Latest TypeScript Package" | ||
description: "Check this if you are using the latest TypeScript package." | ||
options: | ||
- label: "I am using the latest version of TS" | ||
validations: | ||
required: true | ||
- type: checkbox | ||
id: use_latest_typescript_web | ||
attributes: | ||
label: "Use the latest @types/web Package" | ||
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. No "use" but "have tried". Same for the label, options, and the TS version above. |
||
description: "Check this if you are using the latest @types/web package." | ||
Bashamega marked this conversation as resolved.
Show resolved
Hide resolved
|
||
options: | ||
- label: "I am using the latest version of @types/web" | ||
validations: | ||
required: true | ||
- type: textarea | ||
id: additional_context | ||
attributes: | ||
label: "Additional Context" | ||
description: "Any extra information, logs, or references" | ||
placeholder: "Optional details..." |
Uh oh!
There was an error while loading. Please reload this page.