GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | |||||||||||||||||||||||||||||||||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | |||||||||||||||||||||||||||||||||||||||
GeezerVison is showing github.com — chore: remove DigitalOcean from partners by MattIPv4 · Pull Request #9201 · nodejs/nodejs.org Conversation
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
docs/partners.md:7 requires Marketing Team and Technical Steering Committee approval for partner removals, which needs human confirmation.
Review effort: Balanced
Findings: None
Removes DigitalOcean from Node.js partner listings following the end of its Open Source Credits Program.
Changes:
| File | Description |
|---|---|
| README.md | Removes DigitalOcean acknowledgment. |
| packages/ui-components/src/Icons/PartnerLogos/index.ts | Removes DigitalOcean import and export. |
| packages/ui-components/src/Icons/PartnerLogos/DigitalOcean/Logo.tsx | Deletes full logo component. |
| packages/ui-components/src/Icons/PartnerLogos/DigitalOcean/index.ts | Deletes logo component exports. |
| packages/ui-components/src/Icons/PartnerLogos/DigitalOcean/Favicon.tsx | Deletes favicon component. |
| apps/site/public/static/partners/constants.json | Removes DigitalOcean partner metadata. |
| apps/site/next.constants.mjs | Changes hosting comment to refer generically to a VPS. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@ ## main #9201 +/- ## ========================================== - Coverage 86.06% 86.01% -0.05% ========================================== Files 86 86 Lines 6041 6034 -7 Branches 358 358 ========================================== - Hits 5199 5190 -9 - Misses 838 840 +2 Partials 4 4 ☔ View full report in Codecov by Harness. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
be6a1ee to
84849ea
Compare
📦 Build Size ComparisonSummary
|
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
@digitalocean is extending its sponsorship of the build workloads for NodeJS. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Please do not merge this. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
👍 Yep, saw the positive engagement from DigitalOcean overnight in the build issue, will keep following and hopefully we won't need this after all! |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
I know this is draft so shouldn't get merged but in line with what Ryan said I'm putting an explicit block on this for now while the situation is still being negotiated :-)
Sorry, something went wrong.
There was an error while loading. Please reload this page.
84849ea5
Deployed
84849ea5
Deployed
Requested changes must be addressed to merge this pull request.
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.