Skip to content

Conversation

@juliannguyen4
Copy link
Collaborator

@juliannguyen4 juliannguyen4 commented Sep 30, 2025

Wheels build on macOS 14 and pass testing on both github and self-hosted runners using macos 14

@codecov-commenter
Copy link

codecov-commenter commented Sep 30, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.45%. Comparing base (4aea102) to head (5812a4e).
⚠️ Report is 2 commits behind head on dev.

Additional details and impacted files
@@           Coverage Diff           @@
##              dev     #846   +/-   ##
=======================================
  Coverage   82.45%   82.45%           
=======================================
  Files          99       99           
  Lines       14496    14496           
=======================================
  Hits        11952    11952           
  Misses       2544     2544           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@juliannguyen4 juliannguyen4 marked this pull request as ready for review November 18, 2025 16:23
Copy link
Contributor

@justinlee-aerospike justinlee-aerospike left a comment

Choose a reason for hiding this comment

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

looks good

@juliannguyen4 juliannguyen4 merged commit cdf0e0c into dev Nov 18, 2025
155 of 156 checks passed
@juliannguyen4 juliannguyen4 deleted the CLIENT-3793-rm-macos-13-support branch November 18, 2025 16:53
DomPeliniAerospike added a commit that referenced this pull request Nov 18, 2025
* [CLIENT-3793] Remove macOS 13 support (#846)

* Auto-bump version to 18.1.0rc3.dev1 [skip ci]

* [CLIENT-3106] Remove dead code in conversions.c (#817)

- record_to_resultpyobject() was a helper function for client.batch_get_ops(), which is now removed.
- record_to_pyobject_cnvt_list_to_map() and as_list_of_map_to_py_tuple_list(): these were helper functions that were used before Python client version 2.1.3 to return the result of certain map operations. They are no longer used starting from Python client 2.1.3 and higher, so it is safe to remove
- bin_strict_type_checking() isn't used anywhere. But it would be good to consolidate the bin checking code into one place, since it is currently spread out all over the codebase
- as_batch_read_results_to_pyobject() was used by get_many() which has been removed
- batch_read_records_to_pyobject() was used by select_many() which has been removed

Extra Changes

Merge do_*_to_pyobject() methods into their calling methods, since they have the same function signature

* Auto-bump version to 18.1.0rc3.dev2 [skip ci]

---------

Co-authored-by: Julian Nguyen <109386615+juliannguyen4@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
DomPeliniAerospike added a commit that referenced this pull request Nov 18, 2025
* [CLIENT-3793] Remove macOS 13 support (#846)

* Auto-bump version to 18.1.0rc3.dev1 [skip ci]

* [CLIENT-3106] Remove dead code in conversions.c (#817)

- record_to_resultpyobject() was a helper function for client.batch_get_ops(), which is now removed.
- record_to_pyobject_cnvt_list_to_map() and as_list_of_map_to_py_tuple_list(): these were helper functions that were used before Python client version 2.1.3 to return the result of certain map operations. They are no longer used starting from Python client 2.1.3 and higher, so it is safe to remove
- bin_strict_type_checking() isn't used anywhere. But it would be good to consolidate the bin checking code into one place, since it is currently spread out all over the codebase
- as_batch_read_results_to_pyobject() was used by get_many() which has been removed
- batch_read_records_to_pyobject() was used by select_many() which has been removed

Extra Changes

Merge do_*_to_pyobject() methods into their calling methods, since they have the same function signature

* Auto-bump version to 18.1.0rc3.dev2 [skip ci]

---------

Co-authored-by: Julian Nguyen <109386615+juliannguyen4@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
DomPeliniAerospike added a commit that referenced this pull request Nov 18, 2025
* [CLIENT-3793] Remove macOS 13 support (#846)

* Auto-bump version to 18.1.0rc3.dev1 [skip ci]

* [CLIENT-3106] Remove dead code in conversions.c (#817)

- record_to_resultpyobject() was a helper function for client.batch_get_ops(), which is now removed.
- record_to_pyobject_cnvt_list_to_map() and as_list_of_map_to_py_tuple_list(): these were helper functions that were used before Python client version 2.1.3 to return the result of certain map operations. They are no longer used starting from Python client 2.1.3 and higher, so it is safe to remove
- bin_strict_type_checking() isn't used anywhere. But it would be good to consolidate the bin checking code into one place, since it is currently spread out all over the codebase
- as_batch_read_results_to_pyobject() was used by get_many() which has been removed
- batch_read_records_to_pyobject() was used by select_many() which has been removed

Extra Changes

Merge do_*_to_pyobject() methods into their calling methods, since they have the same function signature

* Auto-bump version to 18.1.0rc3.dev2 [skip ci]

---------

Co-authored-by: Julian Nguyen <109386615+juliannguyen4@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

4 participants