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

Update README.md to add the eBPF section #1549

Merged
merged 8 commits into from
Aug 24, 2023

Conversation

r21gh
Copy link
Contributor

@r21gh r21gh commented Aug 10, 2023

I have included the eBPF section to the libraries in this commit. It comprises three distinct Rust libraries, which could be helpful for the project.

@palfrey
Copy link
Collaborator

palfrey commented Aug 12, 2023

So, as noted, redbpf is archived. It may be mentioned in prominent books, but I'm seeing issues like foniod/redbpf#367 and noting this is folks using a tutorial, and now I'm wondering if the book code works at all. Given the archived status, this isn't getting better there anytime soon, unless there's a fork and a new maintainer.

README.md Outdated
@@ -1191,6 +1192,11 @@ See also [About Rust’s Machine Learning Community](https://medium.com/@autumn_

* [serverlesstechnology/cqrs](https://github.com/serverlesstechnology/cqrs) [[cqrs-es](https://crates.io/crates/cqrs-es)] — A framework for CQRS and event sourcing with [user guide](https://doc.rust-cqrs.org/)

### EBPF
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why the capitalisation? Shouldn't this be "eBPF"?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@palfrey eBPF is an unequivocally recognized acronym. Would the linter approve of a sentence that begins with a lowercase letter?

Copy link
Collaborator

Choose a reason for hiding this comment

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

I believe so. It's mostly checking "does the top-level ToC equal the actual sections".

@palfrey palfrey merged commit 72ea7fa into rust-unofficial:main Aug 24, 2023
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.

2 participants