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

Adding Javadoc #1885

Merged
merged 1 commit into from May 22, 2024
Merged

Conversation

Sorna-Sarathi
Copy link

Added the Javadoc for the cleanUp() in CocoaDarkThemeProcessor.

Copy link
Contributor

github-actions bot commented May 13, 2024

Test Results

 1 812 files  ±0   1 812 suites  ±0   1h 29m 35s ⏱️ - 5m 47s
 7 614 tests ±0   7 385 ✅  - 1  228 💤 ±0  1 ❌ +1 
23 994 runs  ±0  23 244 ✅  - 1  749 💤 ±0  1 ❌ +1 

For more details on these failures, see this check.

Results for commit da4a368. ± Comparison against base commit f981728.

♻️ This comment has been updated with latest results.

@elsazac
Copy link
Member

elsazac commented May 21, 2024

To me, these failures seem to be infrastructure related- not related to the PR changes.

Are there known CI issues?

@merks
Copy link
Contributor

merks commented May 21, 2024

@elsazac

Yes, this looks unrelated to the PR itself.

@laeubi

There is only this failure at the end:

image

I can't find anything earlier in the log with more clues...

@laeubi
Copy link
Contributor

laeubi commented May 21, 2024

Looks like a bug in PDE API tools...

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.

This looks good to me

@elsazac
Copy link
Member

elsazac commented May 22, 2024

Given the current freeze period, I will merge these changes once the development cycle reopens, as the test failures are unrelated to the changes.

@mickaelistria
Copy link
Contributor

As mentioned in the "RC1 reminder" mail, "Ongoing changes to documentation, tests or examples do not require approval.”

@mickaelistria mickaelistria merged commit 30bc1dc into eclipse-platform:master May 22, 2024
13 of 16 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.

None yet

5 participants