-
Notifications
You must be signed in to change notification settings - Fork 43
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: up release-please #2305
chore: up release-please #2305
Conversation
2067ce1
to
aa530be
Compare
size-limit report 📦
|
aa530be
to
5cbfd74
Compare
@@ -9,5 +9,7 @@ | |||
"packages/relay": "0.0.16", | |||
"packages/sdk": "0.0.29", | |||
"packages/discovery": "0.0.6", | |||
"packages/react-native-polyfills": "0.0.1" | |||
"packages/react-native-polyfills": "0.0.1", | |||
"packages/sds": "0.0.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@adklempner adding sds
to the release
it's absence was causing failures: release-please wasn't bumping @waku/*
packages in it causing issue with proto package https://github.com/waku-org/js-waku/actions/runs/13945155395/job/39030577038?pr=2309
@@ -1,6 +1,6 @@ | |||
{ | |||
"name": "@waku/rln", | |||
"version": "0.0.1", | |||
"version": "0.1.3", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@danisharora099 fyi rln is getting published too, from where previous repo left
honestly, at this point I think a simple script will be better than release-please |
Problem / Description
release-please
got updatedsds
andrln
packages were not added into the release please hence their deps were not bumped leading to issues https://github.com/waku-org/js-waku/actions/runs/13945155395/job/39030577038?pr=2309Solution
Notes