-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add Regression Test to PR#766 #24
Comments
I propose to increase priority for this issue. The fix has been merged without an automated test and 4.30 release is about to come, so I suggest to provide a (regression) test before release, |
I'm ok with it, I just won't be working on it anytime soon since the changes were small and the functionality has been manually tested so I don't consider this a pressing matter. I removed the assignee, this ticket can be re-prioritized and picked out. |
Waiting for eclipse-platform/eclipse.platform#860 to be merged |
PR has been merged |
Added test is broken: eclipse-platform/eclipse.platform#1075 |
PR created and sent for review (@HeikoKlare ?) |
If possible, add a regression test for the changes in eclipse-platform/eclipse.platform#766. The test should make sure that the current tab is activated (i.e. that its
activate
method is called) when switching between Run Configurations in the Run Configurations Dialog(Moved to eclipse-platform/eclipse.platform#1075)
The text was updated successfully, but these errors were encountered: