Skip to content

Conversation

@edoardop13
Copy link

Hi, I created an LXC script for Proxmox and the maintainer ask me for a link to the project logo.
I sketched these two solutions.

@codecov
Copy link

codecov bot commented Apr 24, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.99%. Comparing base (267f0e5) to head (feebca8).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1049   +/-   ##
=======================================
  Coverage   94.99%   94.99%           
=======================================
  Files          61       61           
  Lines        3397     3397           
=======================================
  Hits         3227     3227           
  Misses        158      158           
  Partials       12       12           
Flag Coverage Δ
unittests 94.99% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@favonia
Copy link
Owner

favonia commented Apr 26, 2025

@edoardop13 Wow, thank you for creating these beautiful logos! I have three comments:

  1. Is it possible to also have scalable versions (e.g., in SVG format, perhaps)?
  2. Do we need text in these logos? I'm worried that the text won't be readable when scaled down.
  3. I feel the circular arrow 🔁 in the first logo (with the sans-serif font) isn't centered; it appears too far to the left to my eyes. Is that intentional? Because of that, I slightly prefer the second one. However, if the alignment is fixed, I might slightly prefer the first one for its flat design.
  4. Are you planning to also create logos for very dark background colors (e.g., "dark mode")?

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