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

update logo to MakieLogos version #4886

Merged
merged 1 commit into from
Mar 22, 2025
Merged

update logo to MakieLogos version #4886

merged 1 commit into from
Mar 22, 2025

Conversation

jkrumbiegel
Copy link
Member

No description provided.

@MakieBot
Copy link
Collaborator

MakieBot commented Mar 22, 2025

Benchmark Results

SHA: bc212f04c3fb1a8ee6d2861e41a79c7628f86d3d

Warning

These results are subject to substantial noise because GitHub's CI runs on shared machines that are not ideally suited for benchmarking.

GLMakie
CairoMakie
WGLMakie

@SimonDanisch
Copy link
Member

Could we maybe directly link the logo from MakieLogos?
Then we just have to update it in one location in the future!

@jkrumbiegel jkrumbiegel reopened this Mar 22, 2025
@jkrumbiegel jkrumbiegel added the skip-changelog Skips changelog enforcer label Mar 22, 2025
@jkrumbiegel
Copy link
Member Author

yeah I guess so, at least the svgs in the readme. The logo.png is used in tests and such

@jkrumbiegel jkrumbiegel merged commit 9e2c93b into master Mar 22, 2025
33 of 41 checks passed
@jkrumbiegel jkrumbiegel deleted the jk/logo-updates branch March 22, 2025 17:38
@jkrumbiegel
Copy link
Member Author

had already updated the refimages so merging this now, but can change to links later as well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip-changelog Skips changelog enforcer
Projects
Status: Merged
Development

Successfully merging this pull request may close these issues.

3 participants