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.

Sign up for GitHub

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

Jump to bottom

[EuiIcon] Run yarn compile-icons #7910

Merged
merged 1 commit into from
Jul 25, 2024
Merged

[EuiIcon] Run yarn compile-icons #7910

merged 1 commit into from
Jul 25, 2024

Conversation

cee-chen
Copy link
Contributor

@cee-chen cee-chen commented Jul 24, 2024 •
edited by mgadewoll
Loading

Summary

@andreadelrio noticed a few icons added in #7873 were not properly optimized/compiled.

All I did in this PR was run yarn compile-icons against main and then update snapshots.

QA

General checklist

N/A, fairly basic tech debt

@cee-chen cee-chen changed the title Optimize icons in main [EuiIcons] Run yarn compile-icons Jul 24, 2024
@cee-chen cee-chen changed the title [EuiIcons] Run yarn compile-icons [EuiIcon] Run yarn compile-icons Jul 24, 2024
@cee-chen cee-chen marked this pull request as ready for review July 24, 2024 17:23
@cee-chen cee-chen requested a review from a team as a code owner July 24, 2024 17:23
@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

@cee-chen cee-chen merged commit f33c787 into elastic:main Jul 25, 2024
11 checks passed
@cee-chen cee-chen deleted the icons-fix branch July 25, 2024 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mgadewoll mgadewoll mgadewoll approved these changes

Assignees
No one assigned
Labels
skip-changelog tech debt
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@cee-chen @elasticmachine @mgadewoll