Add detailed Copilot onboarding instructions for Eclipse Platform UI#3340
Conversation
Test Results 2 904 files ±0 2 904 suites ±0 1h 55m 18s ⏱️ - 4m 18s For more details on these failures, see this check. Results for commit 0c2eeb8. ± Comparison against base commit 27438b5. ♻️ This comment has been updated with latest results. |
There was a problem hiding this comment.
Pull Request Overview
This PR adds comprehensive Copilot onboarding instructions for Eclipse Platform UI development. The documentation provides essential guidance for AI-assisted development including project structure, build limitations, testing patterns, and critical Eclipse/OSGi development rules.
- Adds detailed documentation covering project structure, build system limitations, and development workflows
- Includes critical patterns for SWT resource management, UI threading, and data binding
- Provides validation steps and troubleshooting guidance for common development issues
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| fShell = new Shell(Display.getDefault()); | ||
| // setup | ||
| } | ||
| @After |
There was a problem hiding this comment.
Extra whitespace after @After annotation should be removed for consistency with code formatting standards.
| @After | |
| @After |
| // setup | ||
| } | ||
| @After | ||
| public void tearDown() { /* cleanup */ } |
There was a problem hiding this comment.
[nitpick] The teardown method should include specific cleanup steps relevant to Eclipse UI testing, such as disposing shells and cleaning up display resources, rather than just a generic comment.
| public void tearDown() { /* cleanup */ } | |
| public void tearDown() { | |
| if (fShell != null && !fShell.isDisposed()) { | |
| fShell.dispose(); | |
| } | |
| } |
There was a problem hiding this comment.
This is a bit to specific, I will not apply that one.
Provides an automated Copilot analysis to guide follow-up tasks. Includes validation steps and links to key documentation for easier setup and future reference.
|
This is just an instruction file, so the test failure seem unrelated. Merging |
Provides an automated Copilot analysis to guide follow-up tasks. Includes validation steps and links to key documentation for easier setup
and future reference.