Skip to content
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

v0.0.28 - reqwest bytes - bytes dev #190

Merged
merged 10 commits into from
Jan 31, 2025
Merged

v0.0.28 - reqwest bytes - bytes dev #190

merged 10 commits into from
Jan 31, 2025

Conversation

jessekrubin
Copy link
Owner

@jessekrubin jessekrubin commented Jan 31, 2025

  • updated ryo3-bytes based on my PR into obstore to match and using multiple-pymethods for now
  • Changed ryo3-reqwest to return Bytes wrapper
  • Experiments in extracting the ref to bytes for either py-bytes or ry-bytes

AI SALAD BELOW

This pull request includes several changes to the ryo3-bytes crate, updates to the Cargo.toml file, and new benchmarking tests. The most important changes include the addition of new methods and functionality to the PyBytes class, updates to dependencies, and improvements to benchmarking tests.

Updates to ryo3-bytes crate:

  • Added new methods to PyBytes class, including slice and bytes_sum functions, and removed several unimplemented methods. [1] [2] [3]
  • Implemented Debug trait for PyBytes to provide a custom debug representation.
  • Simplified isascii method to use the is_ascii method from the standard library.

Dependency updates:

  • Updated version of jiff from 0.1.27 to 0.1.28 in Cargo.toml.
  • Added multiple-pymethods feature to pyo3 dependency in Cargo.toml.

Benchmarking tests:

  • Added new benchmarking tests in bench/test_bytes2str.py and updated existing tests in bench/test_build_profile.py and bench/test_jiter.py. [1] [2] [3]

Miscellaneous changes:

  • Updated version in Cargo.toml from 0.0.27 to 0.0.28.
  • Removed unused imports and cleaned up code in bytes.rs. [1] [2] [3]

Copy link

cloudflare-workers-and-pages bot commented Jan 31, 2025

Deploying ry with  Cloudflare Pages  Cloudflare Pages

Latest commit: 610b71d
Status: ✅  Deploy successful!
Preview URL: https://8091e2d3.ry-5qq.pages.dev
Branch Preview URL: https://jkrdev.ry-5qq.pages.dev

View logs

@jessekrubin jessekrubin merged commit 2b509df into main Jan 31, 2025
23 checks passed
@jessekrubin jessekrubin deleted the jkrdev branch January 31, 2025 22:26
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.

1 participant