Skip to content

Conversation

@kevinmichaelchen
Copy link
Owner

No description provided.

…(M1/M2/M3) Macs\n- Update CI workflow to only build for aarch64-apple-darwin\n- Update README to document platform support\n- Other platforms will be added behind a feature flag in future releases
…ect Apple Silicon artifacts\n- Remove other platforms from test matrix
…h dry-run mode\n- Skip npm auth and real publishing on PRs\n- Keep real publishing for tags only
…nd step names\n- Make workflow more visually appealing
…uild job\n- Fix formatting and remove extra newlines
…arget in build command\n- Update target names to match Node.js platform names\n- Add debug output to see binary names
@kevinmichaelchen kevinmichaelchen changed the title feat: Apple Silicon only ci: Fix builds Apr 28, 2025
(reservation.car.pickupLocation == reservation.flight.arrivalAirport ||
reservation.car.pickupLocation == reservation.hotel.address.city) &&
size(reservation.travelers) <= reservation.hotel.maxOccupancy &&
size(reservation.travelers) <= reservation.car.capacity
Copy link
Owner Author

Choose a reason for hiding this comment

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

cel-rust appears to not support the array.min() function that would simplify this

size(res.travelers) <= [a, b].min()

double(reservation.hotel.nightlyRate) * int(reservation.hotel.nights) +
double(reservation.car.dailyRate) * int(reservation.car.days);
let seasonalPrice = basePrice * (timestamp(reservation.hotel.checkIn).getMonth() in [11, 0, 1] ? 1.25 : 1.0);
seasonalPrice * (1.0 - {'BRONZE': 0.05, 'SILVER': 0.10, 'GOLD': 0.15, 'PLATINUM': 0.20}[reservation.customer.loyaltyTier])
Copy link
Owner Author

Choose a reason for hiding this comment

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

don't think CEL supports let

@kevinmichaelchen kevinmichaelchen merged commit f89a663 into main Apr 28, 2025
11 checks passed
@kevinmichaelchen kevinmichaelchen deleted the feat/apple-silicon-only branch April 28, 2025 18:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants