Skip to content

Conversation

maschwenk
Copy link
Contributor

Was noticing this list is kinda hard to parse as-is

@Copilot Copilot AI review requested due to automatic review settings April 8, 2025 03:16
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

README.md:65

  • [nitpick] Consider wrapping 'TS5.8' in backticks for consistent markdown formatting across the table.
| Program creation | done | Same files and module resolution as TS5.8. Not all resolution modes supported yet. |

Comment on lines +66 to +77
| Parsing/scanning | done | Exact same syntax errors as TS5.8 |
| Commandline and `tsconfig.json` parsing | mostly done | Entry point slightly different for now |
| Type resolution | done | Same types as TS5.8 |
| Type checking | done | Same errors, locations, and messages as TS5.8. Types printback in errors may display differently (in progress) |
| JavaScript-specific inference and JS Doc | not ready | - |
| JSX | done | - |
| Declaration emit | not ready | Coming soon |
| Emit (JS output) | in progress | `target: esnext` well-supported, other targets may have gaps |
| Watch mode | prototype | Watches files and rebuilds, but no incremental rechecking |
| Build mode / project references | not ready | - |
| Incremental build | not ready | - |
| Language service (LSP) | prototype | Minimal functionality (errors, hover, go to def). More features coming soon |
Copy link

Copilot AI Apr 8, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[nitpick] Consider standardizing punctuation in the Notes column; for example, either consistently use trailing periods or omit them in all rows.

Suggested change
| Parsing/scanning | done | Exact same syntax errors as TS5.8 |
| Commandline and `tsconfig.json` parsing | mostly done | Entry point slightly different for now |
| Type resolution | done | Same types as TS5.8 |
| Type checking | done | Same errors, locations, and messages as TS5.8. Types printback in errors may display differently (in progress) |
| JavaScript-specific inference and JS Doc | not ready | - |
| JSX | done | - |
| Declaration emit | not ready | Coming soon |
| Emit (JS output) | in progress | `target: esnext` well-supported, other targets may have gaps |
| Watch mode | prototype | Watches files and rebuilds, but no incremental rechecking |
| Build mode / project references | not ready | - |
| Incremental build | not ready | - |
| Language service (LSP) | prototype | Minimal functionality (errors, hover, go to def). More features coming soon |
| Parsing/scanning | done | Exact same syntax errors as TS5.8. |
| Commandline and `tsconfig.json` parsing | mostly done | Entry point slightly different for now. |
| Type resolution | done | Same types as TS5.8. |
| Type checking | done | Same errors, locations, and messages as TS5.8. Types printback in errors may display differently (in progress). |
| JavaScript-specific inference and JS Doc | not ready | - |
| JSX | done | - |
| Declaration emit | not ready | Coming soon. |
| Emit (JS output) | in progress | `target: esnext` well-supported, other targets may have gaps. |
| Watch mode | prototype | Watches files and rebuilds, but no incremental rechecking. |
| Build mode / project references | not ready | - |
| Incremental build | not ready | - |
| Language service (LSP) | prototype | Minimal functionality (errors, hover, go to def). More features coming soon. |

Copilot uses AI. Check for mistakes.

@maschwenk
Copy link
Contributor Author

@microsoft-github-policy-service agree

@maschwenk
Copy link
Contributor Author

@RyanCavanaugh do I need to add anything else before merge?

@RyanCavanaugh RyanCavanaugh added this pull request to the merge queue Apr 8, 2025
Merged via the queue into microsoft:main with commit 74c01ca Apr 8, 2025
23 checks passed
zshannon pushed a commit to zshannon/typescript-go that referenced this pull request Oct 6, 2025
zshannon pushed a commit to zshannon/typescript-go that referenced this pull request Oct 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants