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 network plugin #19

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from
Draft

feat: add network plugin #19

wants to merge 7 commits into from

Conversation

Pantani
Copy link
Contributor

@Pantani Pantani commented Oct 15, 2023

move network plugin to this repo

@Pantani Pantani self-assigned this Oct 15, 2023
@Pantani Pantani marked this pull request as ready for review November 8, 2023 12:53
Copy link
Member

@ilgooz ilgooz left a comment

Choose a reason for hiding this comment

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

Thanks for pushing this!

Meanwhile, we should make it clear that this code originates from another repo and should give attributions to the other authors.

We should try finding the best way of giving attributions; at very least we can include them as co-authors by pushing an empty commit. And in the description of the commit we could add more information as/if needed.

Please see the format:

image

@Pantani Pantani marked this pull request as draft February 1, 2024 19:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

None yet

3 participants