Skip to content

Add basic code mining tracing option to trace rendering #2788

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

iloveeclipse
Copy link
Member

See #2786

@iloveeclipse
Copy link
Member Author

@mickaelistria, @tobiasmelcher : WDYT? At least some basic data?

@mickaelistria
Copy link
Contributor

I don't think it's so relevant to add this in the code; adding those as conditional breakpoints when needed is easy enough IMO.

@iloveeclipse
Copy link
Member Author

I was thinking about people who don't have SDK sources checked out - these options are tracing options everyone can set at runtime.

Copy link
Contributor

Test Results

 1 214 files   -   607   1 214 suites   - 607   1h 7m 23s ⏱️ - 20m 30s
 7 723 tests ±    0   7 490 ✅  -     5  233 💤 +  5  0 ❌ ±0 
16 220 runs   - 8 110  15 705 ✅  - 7 876  515 💤  - 234  0 ❌ ±0 

Results for commit e2ac351. ± Comparison against base commit c9f129d.

This pull request skips 5 tests.
org.eclipse.ui.genericeditor.tests.HoverTest ‑ testEnabledWhenHover
org.eclipse.ui.genericeditor.tests.HoverTest ‑ testMultipleHover
org.eclipse.ui.genericeditor.tests.HoverTest ‑ testProblemHover
org.eclipse.ui.genericeditor.tests.HoverTest ‑ testSingleHover
org.eclipse.ui.genericeditor.tests.ShowInformationTest ‑ testInformationControl

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