Skip to content
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

Handle updating "Difference/s" label on compare editor for empty toolbar #1319

Conversation

lathapatil
Copy link
Contributor

illegalArgumentException is handled when toolbar contributions are empty while updating the "Difference/s" label on toolbar of compare editor.

Fixes : #1295

@lathapatil
Copy link
Contributor Author

@iloveeclipse please review the code.

Copy link
Member

@iloveeclipse iloveeclipse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, looks good

Copy link
Contributor

github-actions bot commented Apr 17, 2024

Test Results

   639 files  ±0     639 suites  ±0   44m 16s ⏱️ + 4m 2s
 3 949 tests ±0   3 927 ✅ +1   22 💤 ±0  0 ❌  - 1 
12 453 runs  ±0  12 292 ✅ +1  161 💤 ±0  0 ❌  - 1 

Results for commit b95364f. ± Comparison against base commit bcb197c.

♻️ This comment has been updated with latest results.

Copy link
Contributor

@fedejeanne fedejeanne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indentation is wrong

@fedejeanne fedejeanne force-pushed the Bugs/1295_illegalArgumentExceptiom branch from 1ab4db3 to b95364f Compare April 17, 2024 11:00
Co-authored-by: Federico Jeanne <2205684+fedejeanne@users.noreply.github.com>
@fedejeanne fedejeanne force-pushed the Bugs/1295_illegalArgumentExceptiom branch from b95364f to 710552c Compare April 17, 2024 11:44
@fedejeanne fedejeanne merged commit 8d37020 into eclipse-platform:master Apr 17, 2024
7 of 10 checks passed
fedejeanne added a commit to fedejeanne/eclipse.platform that referenced this pull request Apr 17, 2024
fedejeanne added a commit that referenced this pull request Apr 17, 2024
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.

IllegalArgumentException: Group not found: diffLabel
3 participants