Skip to content

feat: Add TuxCare partner#8973

Merged
bmuenzenmeyer merged 5 commits into
mainfrom
tuxcare
Jun 25, 2026
Merged

feat: Add TuxCare partner#8973
bmuenzenmeyer merged 5 commits into
mainfrom
tuxcare

Conversation

@bjohansebas

Copy link
Copy Markdown
Member

closes #8971

cc: @nodejs/marketing

Description

Validation

Related Issues

Check List

  • I have read the Contributing Guidelines and made commit messages that follow the guideline.
  • I have run pnpm format to ensure the code follows the style guide.
  • I have run pnpm test to check if all tests are passing.
  • I have run pnpm build to check if the website builds without errors.
  • I've covered new added functionality with unit tests if necessary.

- Created a new directory for TuxCare logos.
- Imported Favicon and Logo components from TuxCare directory.
- Updated the main PartnerLogos index to include TuxCare.
Copilot AI review requested due to automatic review settings June 24, 2026 22:59
@bjohansebas bjohansebas requested a review from a team as a code owner June 24, 2026 22:59

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@cursor

cursor Bot commented Jun 24, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Content and static partner/logo additions only; no auth, runtime, or data-handling changes.

Overview
Adds TuxCare as an OpenJS Ecosystem Sustainability Program partner on the site, with branding assets and EOL commercial-support copy.

TuxCare is registered in constants.json (esp category) and wired through PartnerLogos with new Favicon and Logo SVG components so it appears wherever partners are listed (e.g. partner grids keyed by partner.id).

The End-Of-Life page now names TuxCare alongside HeroDevs and NodeSource, and adds a short blurb for TuxCare Endless Lifecycle Support (ELS) with a link to their offering.

Also fixes React-correct SVG props on the Node.js logo (clipPath, floodColor, floodOpacity).

Reviewed by Cursor Bugbot for commit 74c911b. Bugbot is set up for automated code reviews on this repo. Configure here.

@vercel

vercel Bot commented Jun 24, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
nodejs-org Ready Ready Preview Jun 25, 2026 4:38pm

Request Review

@bjohansebas

Copy link
Copy Markdown
Member Author
imagen imagen

@codecov

codecov Bot commented Jun 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.36%. Comparing base (71312e2) to head (74c911b).
⚠️ Report is 2 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8973      +/-   ##
==========================================
- Coverage   75.39%   75.36%   -0.04%     
==========================================
  Files          98       98              
  Lines        8630     8637       +7     
  Branches      318      319       +1     
==========================================
+ Hits         6507     6509       +2     
- Misses       2119     2124       +5     
  Partials        4        4              

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

Comment thread apps/site/pages/en/about/eol.mdx Outdated
Co-authored-by: Jordan Harband <ljharb@gmail.com>
Signed-off-by: Sebastian Beltran <bjohansebas@gmail.com>

@bmuenzenmeyer bmuenzenmeyer left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My only nit with that listing of Node.js versions is that as 22 becomes EOL, and beyond, it will need to be updated. I suppose individual ELS versions are product feature that they may choose to do or not - I'm just pointing out a maintenance burden on our end due to the copy.

@bmuenzenmeyer

Copy link
Copy Markdown
Contributor

i didn't look at the brand kit but i wondered if the tuxcare logo could be made bigger by consuming some of its vertical whitespace padding.

@kyliewd

kyliewd commented Jun 25, 2026

Copy link
Copy Markdown

My only nit with that listing of Node.js versions is that as 22 becomes EOL, and beyond, it will need to be updated. I suppose individual ELS versions are product feature that they may choose to do or not - I'm just pointing out a maintenance burden on our end due to the copy.

I am fine with removing the version numbers

@bmuenzenmeyer

Copy link
Copy Markdown
Contributor

i didn't look at the brand kit but i wondered if the tuxcare logo could be made bigger by consuming some of its vertical whitespace padding.

image

making it 80px high aligns it's size visually with herodevs

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Want fixes drafted automatically? Bugbot Autofix can create code changes for findings. A team admin can enable Autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit c853a38. Configure here.

Comment thread packages/ui-components/src/Icons/PartnerLogos/TuxCare/Logo.tsx
@bjohansebas

Copy link
Copy Markdown
Member Author

LGTM! Do we want to fast-track this?

@bmuenzenmeyer bmuenzenmeyer added the fast-track Fast Tracking PRs label Jun 25, 2026
@bmuenzenmeyer

Copy link
Copy Markdown
Contributor

requesting fast track - citing https://openjsf.org/blog/tuxcare-joins-openjs-foundation

@bmuenzenmeyer

Copy link
Copy Markdown
Contributor

a note - the black only tuxcare logo, was that chosen intentionally? their website has a black and blue one

@kyliewd

kyliewd commented Jun 25, 2026

Copy link
Copy Markdown

if you need a different color let me know

@bjohansebas

Copy link
Copy Markdown
Member Author

It was because that's how it appeared in the files Aviv sent me, since I was never able to access their website.

Comment thread apps/site/pages/en/about/eol.mdx Outdated
@bmuenzenmeyer

Copy link
Copy Markdown
Contributor

I have all of them in the zip - @kyliewd do we or they have a preference?

image

@kyliewd

kyliewd commented Jun 25, 2026

Copy link
Copy Markdown

whichever one will look better on the website, but we can default to the blue one if it doesnt matter since that is their main one.

@bmuenzenmeyer

Copy link
Copy Markdown
Contributor

oh, actually i just noticed we support both based on the color scheme of the user - perhaps thats fine. if they want to change it they can ask or contribute

Co-authored-by: Sebastian Beltran <bjohansebas@gmail.com>
Signed-off-by: Brian Muenzenmeyer <brian.muenzenmeyer@gmail.com>
@bmuenzenmeyer bmuenzenmeyer enabled auto-merge June 25, 2026 16:43
@bmuenzenmeyer bmuenzenmeyer added this pull request to the merge queue Jun 25, 2026
Merged via the queue into main with commit 1c3273c Jun 25, 2026
15 checks passed
@bmuenzenmeyer bmuenzenmeyer deleted the tuxcare branch June 25, 2026 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fast-track Fast Tracking PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Edit to the Node.js EOL Page

6 participants