Closed benjaminpkane closed 1 month ago
The changes introduce a new step in the GitHub Actions workflow to upload a wheel artifact conditionally based on the platform. Additionally, the setup.py
script for the package/db
has been updated to include MongoDB download links for CentOS 7, 8, and 9, updated links for Debian 9, and an additional link for Windows (amd64). The version has also been incremented to 1.1.3.
File | Change Summary |
---|---|
.github/workflows/build-db.yml | Added a conditional step for uploading a wheel artifact based on the platform being different from 'sdist'. |
package/db/setup.py | Added MongoDB download links for CentOS 7, 8, and 9, updated Debian 9 links, added a Windows (amd64) link, and incremented the version to 1.1.3. |
Objective (Issue #) | Addressed | Explanation |
---|---|---|
[#4427] Ensure MongoDB can be installed on CentOS 7 | ✅ | |
[#4427] Add MongoDB download links for CentOS 8 and 9, and update Debian 9 links | ✅ | |
[#4427] Increment version to reflect changes | ✅ |
In the code, a change so bright,
MongoDB links take flight,
CentOS, Debian, Windows too,
Now the setup's fresh and new.
FiftyOne, without a hitch,
On platforms, it will swiftly stitch.
Version up, the future's clear,
With every fix, we hold it dear.
[!WARNING]
Review ran into problems
Problems (1)
* Git: Failed to clone repository. Please contact CodeRabbit support.
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 16.00%. Comparing base (
1688ce2
) to head (ec57f0d
). Report is 1031 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
What changes are proposed in this pull request?
Add CentOS entries to the
fiftyone-db
package. CentOS uses the same downloads as RHEL. See MongoDB downloads for confirmationResolves #4427
How is this patch tested? If it is not, please explain why.
Using CentOS
What areas of FiftyOne does this PR affect?
fiftyone
Python library changesSummary by CodeRabbit
New Features
Updates
CI/CD