Skip to content

Add SVGs to org.eclipse.search #2912

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

Merged

Conversation

Michael5601
Copy link
Contributor

This PR adds SVGs for all icons in the bundle org.eclipse.search.

See also this PR for more information.

@Michael5601 Michael5601 force-pushed the org.eclipse.search.SVGs branch 2 times, most recently from ec20192 to 1ecbfe0 Compare April 15, 2025 11:58
Copy link
Contributor

github-actions bot commented Apr 15, 2025

Test Results

 1 824 files  ±0   1 824 suites  ±0   1h 40m 32s ⏱️ + 7m 56s
 7 918 tests ±0   7 690 ✅ ±0  228 💤 ±0  0 ❌ ±0 
23 841 runs  ±0  23 093 ✅ ±0  748 💤 ±0  0 ❌ ±0 

Results for commit c37cb17. ± Comparison against base commit 73ea7e9.

♻️ This comment has been updated with latest results.

@HeikoKlare HeikoKlare force-pushed the org.eclipse.search.SVGs branch from 1ecbfe0 to 5864b93 Compare April 17, 2025 14:49
This commit adds SVGs for all icons in the bundle `org.eclipse.search`.
@HeikoKlare HeikoKlare force-pushed the org.eclipse.search.SVGs branch from 5864b93 to c37cb17 Compare April 18, 2025 16:36
@HeikoKlare
Copy link
Contributor

Merging this despite Jenkins failure caused by https://gitlab.eclipse.org/eclipsefdn/helpdesk/-/issues/5892

Since master builds seem to still work on Jenkins, the subsequent master build should hopefully work, thus I will check results of that build to retrospectively validate this PR.

@HeikoKlare HeikoKlare merged commit 7302fcc into eclipse-platform:master Apr 22, 2025
11 of 13 checks passed
@Michael5601 Michael5601 deleted the org.eclipse.search.SVGs branch April 22, 2025 09:37
@HeikoKlare
Copy link
Contributor

No new issues on subsequent master build for 7302fcc

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