-
Notifications
You must be signed in to change notification settings - Fork 111
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[CLIENT-2217] macOS wheels: update OpenSSL dependency from 1.1.1 to 3 #471
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #471 +/- ##
==========================================
- Coverage 80.89% 80.87% -0.02%
==========================================
Files 100 100
Lines 15053 15047 -6
==========================================
- Hits 12177 12170 -7
- Misses 2876 2877 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
957b2a5
to
a87ca89
Compare
21af794
to
de6779c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is maybe a discrepancy in the MD file, otherwise it looks okay.
BUILD.md
Outdated
@@ -87,8 +87,7 @@ By default macOS will be missing command line tools. | |||
|
|||
The dependencies can be installed through the macOS package manager [Homebrew](http://brew.sh/). | |||
|
|||
brew install openssl@1 | |||
# brew uninstall openssl@3 | |||
brew uninstall openssl@3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do you mean 'install', not 'uninstall'?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
change looks good
macOS arm64 and x86_64 wheels build and pass testing