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

chore(release): update changelog for v0.20.0 release #2026

Merged
merged 4 commits into from
Sep 14, 2023

Conversation

vpavlin
Copy link
Member

@vpavlin vpavlin commented Sep 12, 2023

Description

Changelog for v0.20.0

Please review carefully, suggest highlights or upgrade instructions (we should probably emphasize RLN no longer being experimental and maybe point to RLN docs in upgrade instructions?)

@rymnc I meantioned you in highlights because you flagged the issue with release-notes label, but I am not sure what specifically would you like to highlight:)

Changes

  • updated changelog

CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated
* RLN is now part of standard release (is no longer EXPERIMENTAL feature)
* Interop tests between nwaku and js-waku are now gating PRs and releases
* Libwaku has been made threadsafe
* Autosharding being implementated across various protocols
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* Autosharding being implementated across various protocols
* Made various protocol implementations use autosharding.

Copy link
Contributor

Choose a reason for hiding this comment

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

I want it to be clear that the protocols did not change only the impl. Not sure if it's clearer but 🤷

Copy link
Contributor

Choose a reason for hiding this comment

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

How about just * Added autosharding option on various protocol APIs. This is simply a change to make the API (internal/external) more flexible

Copy link
Contributor

@jm-clius jm-clius left a comment

Choose a reason for hiding this comment

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

This is a great release. Thanks!

CHANGELOG.md Outdated
* RLN is now part of standard release (is no longer EXPERIMENTAL feature)
* Interop tests between nwaku and js-waku are now gating PRs and releases
* Libwaku has been made threadsafe
* Autosharding being implementated across various protocols
Copy link
Contributor

Choose a reason for hiding this comment

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

How about just * Added autosharding option on various protocol APIs. This is simply a change to make the API (internal/external) more flexible

CHANGELOG.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@Ivansete-status Ivansete-status left a comment

Choose a reason for hiding this comment

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

LGTM! I just added one comment. Thanks!

CHANGELOG.md Outdated Show resolved Hide resolved
vpavlin and others added 2 commits September 13, 2023 14:09
Co-authored-by: Ivan Folgueira Bande <128452529+Ivansete-status@users.noreply.github.com>
Co-authored-by: Simon-Pierre Vivier <simvivier@status.im>
CHANGELOG.md Outdated
* Interop tests between nwaku and js-waku are now gating PRs and releases
* Libwaku has been made more threadsafe (1 out of 3 improvements applied.)
* Added autosharding option on various protocol APIs
* TODO: @rymnc https://github.com/waku-org/nwaku/pull/1966
Copy link
Contributor

Choose a reason for hiding this comment

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

@vpavlin here we need to write that the rln-relay-membership-group-index flag has been removed, along with rln-relay-pubsub-topic and rln-relay-content-topic

Copy link
Contributor

@rymnc rymnc left a comment

Choose a reason for hiding this comment

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

LGTM, added a comment on the todo

Copy link
Contributor

@NagyZoltanPeter NagyZoltanPeter left a comment

Choose a reason for hiding this comment

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

LGTM!
I would not advertise Legacy Filter's Rest API support - so it's good as is.

@gabrielmer
Copy link
Contributor

Looks great :) Thanks!

Copy link
Contributor

@alrevuelta alrevuelta left a comment

Choose a reason for hiding this comment

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

lgtm!

@vpavlin vpavlin merged commit 9085b1b into master Sep 14, 2023
7 checks passed
@vpavlin vpavlin deleted the chore/changelog-v0.20.0 branch September 14, 2023 14:03
vpavlin added a commit that referenced this pull request Sep 14, 2023
* chore(release): update changelog for v0.20.0 release

* Update CHANGELOG.md

Co-authored-by: Ivan Folgueira Bande <128452529+Ivansete-status@users.noreply.github.com>

* Update CHANGELOG.md

Co-authored-by: Simon-Pierre Vivier <simvivier@status.im>

* fix typo and autosharding highlight, note CLI options removal

---------

Co-authored-by: Ivan Folgueira Bande <128452529+Ivansete-status@users.noreply.github.com>
Co-authored-by: Simon-Pierre Vivier <simvivier@status.im>
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

8 participants