Skip to content

Commit ca3a056

Browse files
Copilotjakebailey
andauthored
Add guidance to use @strict: true by default in new compiler tests (#1480)
Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: jakebailey <[email protected]>
1 parent 5d60697 commit ca3a056

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

.github/copilot-instructions.md

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -56,8 +56,10 @@ export interface Person {
5656
function greet(person) {
5757
console.log(`Hello, ${person.name}!`);
5858
}
59-
```
60-
59+
```
60+
61+
**New compiler tests should always enable strict mode (`@strict: true`) unless the bug specifically involves non-strict mode behavior.**
62+
6163
Tests don't always need the above `@option`s specified, but they are common to specify or modify.
6264
Tests can be run with multiple settings for a given option by using a comma-separated list (e.g. `@option: settingA,settingB`).
6365
`@filename` is only required when a test has multiple files, or when writing a test for a single JavaScript file (where `allowJs` or `checkJs` is enabled).

0 commit comments

Comments
 (0)