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

feat: Add set for Array<T> #1005

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

fslongjin
Copy link

Add set() function for Array<T>

Copy link

netlify bot commented Jul 30, 2024

Deploy Preview for aya-rs-docs ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 4f0b904
🔍 Latest deploy log https://app.netlify.com/sites/aya-rs-docs/deploys/66a8db710145b20008ae2721
😎 Deploy Preview https://deploy-preview-1005--aya-rs-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@mergify mergify bot added the aya-bpf This is about aya-bpf (kernel) label Jul 30, 2024
@fslongjin
Copy link
Author

ping

@tamird
Copy link
Member

tamird commented Aug 26, 2024

ping

Needs tests.

@vadorovsky
Copy link
Member

To be precise - a great place to add tests would be https://github.com/aya-rs/aya/blob/main/test/integration-ebpf/src/map_test.rs

A simple set call on any kind of map with some dummy values would be enough 🙂

@fslongjin
Copy link
Author

Ok, I'll add tests tomorrow

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aya-bpf This is about aya-bpf (kernel) needs tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants