Skip to content

Conversation

@sgammon
Copy link
Member

@sgammon sgammon commented Jul 19, 2025

Ready for review Powered by Pull Request Badge

Summary

Fixes Rust's macOS minimum version target, which was being set to the host version; it is now set to 15.0, aligning with our CFLAGS.

Note that 15.0 may not be the optimal version we should land on, but now at least it's consistent.

@sgammon sgammon added this to the Release R18: Beta milestone Jul 19, 2025
@sgammon sgammon requested a review from a team July 19, 2025 05:16
@sgammon sgammon self-assigned this Jul 19, 2025
@sgammon sgammon added bug Something isn't working platform:macos Issues relating to macOS labels Jul 19, 2025
@sgammon sgammon added this to Elide Jul 19, 2025
@sgammon sgammon moved this to Done in Elide Jul 19, 2025
fix: macos minimum value isn't consistent
fix: missing libs for macos

Signed-off-by: Sam Gammon <sam@elide.ventures>
@sgammon sgammon force-pushed the fix/macos-min-version-rust branch from 32a5e05 to 3b6c010 Compare July 19, 2025 05:49
@codecov
Copy link

codecov bot commented Jul 19, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 39.87%. Comparing base (4b4794b) to head (3b6c010).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1550   +/-   ##
=======================================
  Coverage   39.87%   39.87%           
=======================================
  Files         768      768           
  Lines       36941    36941           
  Branches     5218     5218           
=======================================
  Hits        14730    14730           
  Misses      20446    20446           
  Partials     1765     1765           
Flag Coverage Δ
jvm 39.87% <ø> (ø)
lib 39.87% <ø> (ø)

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


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4b4794b...3b6c010. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@sgammon sgammon merged commit c0c60e6 into main Jul 19, 2025
17 checks passed
@elidebot elidebot mentioned this pull request Jul 19, 2025
29 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working platform:macos Issues relating to macOS

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants