Skip to content

Commit c412462

Browse files
committed
Merge branch 'main' into pr/mohamedamara1/6835
1 parent 41842ab commit c412462

File tree

364 files changed

+25691
-6038
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

364 files changed

+25691
-6038
lines changed

.eslintignore

Lines changed: 0 additions & 5 deletions
This file was deleted.

.eslintrc.base.json

Lines changed: 0 additions & 262 deletions
This file was deleted.

.eslintrc.browser.json

Lines changed: 0 additions & 9 deletions
This file was deleted.

.eslintrc.json

Lines changed: 0 additions & 10 deletions
This file was deleted.

.eslintrc.node.json

Lines changed: 0 additions & 9 deletions
This file was deleted.

.eslintrc.webviews.json

Lines changed: 0 additions & 9 deletions
This file was deleted.

.github/ISSUE_TEMPLATE/bug_report.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ about: Create a report to help us improve
1111
- VSCode Version:
1212
- OS:
1313
- Repository Clone Configuration (single repository/fork of an upstream repository):
14-
- Github Product (Github.com/Github Enterprise version x.x.x):
14+
- GitHub Product (GitHub.com/GitHub Enterprise version x.x.x):
1515

1616
Steps to Reproduce:
1717

.github/copilot-instructions.md

Lines changed: 32 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,32 @@
1+
## General Guidelines
2+
- **Follow the existing code style**: Use single quotes, semicolons, and 2-space indentation. See `.eslintrc.base.json` for detailed linting rules.
3+
- **TypeScript**: Use modern TypeScript features. Prefer explicit types where clarity is needed, but do not over-annotate.
4+
- **React/JSX**: Webviews use React-style JSX with custom factories (`vscpp`, `vscppf`).
5+
- **Strictness**: Some strictness is disabled in `tsconfig.base.json` (e.g., `strictNullChecks: false`), but new code should avoid unsafe patterns.
6+
- **Testing**: Place tests under `src/test`. Do not include test code in production files.
7+
- **Localization**: Use `%key%` syntax for strings that require localization. See `package.nls.json`.
8+
- **Regular Expressions**: Use named capture groups for clarity when working with complex regex patterns.
9+
10+
## Extension-Specific Practices
11+
- **VS Code API**: Use the official VS Code API for all extension points. Register commands, views, and menus via `package.json`.
12+
- **Configuration**: All user-facing settings must be declared in `package.json` under `contributes.configuration`.
13+
- **Activation Events**: Only add new activation events if absolutely necessary.
14+
- **Webviews**: Place webview code in the `webviews/` directory. Use the shared `common/` code where possible.
15+
- **Commands**: Register new commands in `package.json` and implement them in `src/commands.ts` or a relevant module.
16+
- **Logging**: Use the `Logger` utility for all logging purposes. Don't use console.log or similar methods directly.
17+
- **Test Running**: Use tools over tasks over scripts to run tests.
18+
19+
## Specific Feature Practices
20+
- **Commands**: When adding a new command, consider whether it should be available in the command palette, context menus, or both. Add the appropriate menu entries in `package.json` to ensure the command is properly included, or excluded (command palette), from menus.
21+
22+
## Pull Request Guidelines
23+
- Never touch the yarn.lock file.
24+
- Run `yarn run lint` and also `npm run hygiene` and fix any errors or warnings before committing.
25+
26+
## Testing
27+
- Use `describe` and `it` blocks from Mocha for structuring tests.
28+
- Tests MUST run product code.
29+
- Tests should be placed near the code they are testing, in a `test` folder. Name test fies with a `.test.ts` suffix.
30+
31+
---
32+
_Last updated: 2025-06-20_

0 commit comments

Comments
 (0)