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

README: update to reflect the project's situation in H2 2022 #804

Merged
merged 1 commit into from Sep 28, 2022

Conversation

ti-mo
Copy link
Collaborator

@ti-mo ti-mo commented Sep 26, 2022

  • 'Officially' mention 'ebpf-go' as being a thing.
  • Dropped the CF/Cilium notice as the project has received frequent and significant contributions from the wider community and was only partially relevant anymore. We no longer need it to convey the legitimacy of the project.
  • Explicitly call out GH Discussions, which is has seen significant use so far.

- 'Officially' mention 'ebpf-go' as being a thing
- Dropped the CF/Cilium notice as the project has received frequent
  and significant contributions from the wider community and was only
  partially relevant anymore.
- Explicitly call out GH Discussions, which is has seen significant
  use so far.

Signed-off-by: Timo Beckers <timo@isovalent.com>
@ti-mo ti-mo requested a review from lmb September 26, 2022 11:51
@@ -4,14 +4,12 @@

![HoneyGopher](.github/images/cilium-ebpf.png)

eBPF is a pure Go library that provides utilities for loading, compiling, and
ebpf-go is a pure Go library that provides utilities for loading, compiling, and
Copy link
Collaborator

Choose a reason for hiding this comment

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

Clear as mud? ;D https://pkg.go.dev/github.com/dropbox/goebpf Referring to it as ebpf-go when speaking about it would work for me, but I don't want to rename the package ebpf-go since that forces import renaming.

Copy link
Collaborator Author

@ti-mo ti-mo Sep 26, 2022

Choose a reason for hiding this comment

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

Of course not, no import renaming. :)

ebpf-go is unambiguous and short. The dropbox one sounds like göbpf in my head, so no confusion possible.

@ti-mo ti-mo marked this pull request as ready for review September 28, 2022 07:43
@ti-mo ti-mo merged commit d0f3bfb into cilium:master Sep 28, 2022
@ti-mo ti-mo deleted the tb/update-readme branch September 28, 2022 11:17
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.

None yet

2 participants