-
Notifications
You must be signed in to change notification settings - Fork 286
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
aya: Don't store btf_fd in MapData #702
Conversation
✅ Deploy Preview for aya-rs-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
LGTM. Would be preferable to squash these two commits, imo.
6b49ec5
to
58498c5
Compare
Hey @alessandrod, this pull request changes the Aya Public API and requires your review. |
Related to the api changes: #703 |
@dave-tucker, this pull request is now in conflict and requires a rebase. |
58498c5
to
bc60ad7
Compare
This is only used in create and therefore can be passed as a parameter. Signed-off-by: Dave Tucker <dave@dtucker.co.uk>
bc60ad7
to
db975e9
Compare
This is only used in create and therefore can be passed
as a parameter.