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

Remove reference to unmaintained p2p networking library #34172

Merged
merged 1 commit into from
Jun 15, 2024
Merged

Remove reference to unmaintained p2p networking library #34172

merged 1 commit into from
Jun 15, 2024

Conversation

fjij
Copy link
Contributor

@fjij fjij commented Jun 14, 2024

Description

Remove reference to a defunct library p2p so it doesn't mislead readers. The library hasn't worked for years, and is not being maintained.

Motivation

I was reading the article and tried to use this library, but according to the issues it is no longer being maintained and does not work as of 2018.

Additional details

Issue that mentions it does not work and is not being maintained:

js-platform/p2p#24

The library hasn't worked for years. See this 2018 issue: js-platform/p2p#24.
@fjij fjij requested a review from a team as a code owner June 14, 2024 18:42
@fjij fjij requested review from hamishwillee and removed request for a team June 14, 2024 18:42
@github-actions github-actions bot added Content:Games Games docs size/s [PR only] 6-50 LoC changed labels Jun 14, 2024
Copy link
Member

@estelle estelle left a comment

Choose a reason for hiding this comment

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

Thank you. Also, congratulations on your first merged MDN content PR. Welcome to the team!

@estelle estelle merged commit 1807880 into mdn:main Jun 15, 2024
7 checks passed
Copy link
Contributor

@fjij fjij deleted the patch-1 branch June 17, 2024 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:Games Games docs size/s [PR only] 6-50 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants