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

docs: add migration guide for libp2p@0.46.x #1883

Merged
merged 7 commits into from
Jul 28, 2023

Conversation

achingbrain
Copy link
Member

Adds migration guide detailing breaking changes in libp2p@0.46.x

Adds migration guide detailing breaking changes in libp2p@0.46.x
Copy link
Member

@maschad maschad left a comment

Choose a reason for hiding this comment

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

Should we also include the option to disable runOnConnectionOpen for identify queries that was added in #1784 ?

doc/migrations/v0.45-v0.46.md Show resolved Hide resolved

**Before**

```js
Copy link
Member

Choose a reason for hiding this comment

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

Going forward we should really utilize Typescript examples so that they can be compiled and thus reduce the probability of doc errors.

Copy link
Member Author

Choose a reason for hiding this comment

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

Probably, though it'll involve a lot of boilerplate imports.

Copy link
Member

Choose a reason for hiding this comment

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

True, but otherwise we lose the benefit of the type safety. Perhaps we can begin with #1777 as a precedent.

@achingbrain
Copy link
Member Author

Should we also include the option to disable runOnConnectionOpen for identify queries

I've omitted it here because it's not a breaking change, though I suppose it wouldn't hurt to include new features too.

Copy link
Member

@maschad maschad left a comment

Choose a reason for hiding this comment

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

We should also enable doc-check on this doc if we are going to have TS snippets.

doc/migrations/v0.45-v0.46.md Outdated Show resolved Hide resolved
doc/migrations/v0.45-v0.46.md Outdated Show resolved Hide resolved
@achingbrain achingbrain merged commit 69c93ac into master Jul 28, 2023
17 checks passed
@achingbrain achingbrain deleted the docs/add-migration-guide-for-46 branch July 28, 2023 17:15
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