Add openEuler OS support for RPM-based distributions#1929
Add openEuler OS support for RPM-based distributions#1929JunchengDwain wants to merge 1 commit intonodesource:masterfrom
Conversation
- Added /etc/openEuler-release file check in RPM detection logic - Modified all RPM setup scripts (setup_*.x) and base_script.sh - Added openEuler to GitHub Actions CI workflow (rpm job only): - openeuler/openeuler:22.03-lts-sp4 - openeuler/openeuler:24.03-lts-sp3 - Note: openEuler has no minimal variant, so not added to rpm-minimal job
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (12)
WalkthroughThis pull request extends OpenEuler OS support across Node.js distribution installation scripts. Two OpenEuler container images were added to CI testing workflows, and RPM system detection logic was updated across all setup scripts to recognize OpenEuler systems as valid RPM-based environments alongside existing Red Hat and Amazon Linux checks. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary by CodeRabbit
New Features
Chores