Skip to content

Conversation

@deepika-u
Copy link
Contributor

No description provided.

@deepika-u
Copy link
Contributor Author

@vogella : When you get some time can you take a look at this please.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 16, 2025

Test Results

 3 018 files  ±0   3 018 suites  ±0   2h 16m 57s ⏱️ + 5m 19s
 8 248 tests ±0   8 000 ✅ ±0  248 💤 ±0  0 ❌ ±0 
23 664 runs  ±0  22 873 ✅ ±0  791 💤 ±0  0 ❌ ±0 

Results for commit 5ae934b. ± Comparison against base commit 60a8a9c.

♻️ This comment has been updated with latest results.

@vogella
Copy link
Contributor

vogella commented Dec 17, 2025

@elsazac can you review? You are also active in clean-ups

@elsazac
Copy link
Member

elsazac commented Dec 17, 2025

@elsazac can you review? You are also active in clean-ups

yes, will take a look at this.

Copy link
Member

@elsazac elsazac left a comment

Choose a reason for hiding this comment

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

LGTM.

@vogella
Copy link
Contributor

vogella commented Dec 17, 2025

@elsazac in this case please merge

@vogella
Copy link
Contributor

vogella commented Dec 18, 2025

Merging based on the review of @elsazac

Thanks @deepika-u and @elsazac

@vogella vogella merged commit a629637 into eclipse-platform:master Dec 18, 2025
18 checks passed
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.

3 participants