-
Notifications
You must be signed in to change notification settings - Fork 8
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(deps): update dependency ser-kit to v1 #321
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Deploying zodiac-pilot with Cloudflare Pages
|
Deploying zodiac-pilot-connect with Cloudflare Pages
|
51217c1
to
ab2e360
Compare
ab2e360
to
968dd0a
Compare
968dd0a
to
fc73c1c
Compare
fc73c1c
to
2bf8861
Compare
2bf8861
to
c2f1ee3
Compare
@cristovaoth, I would appreciate it if you could have a quick look over the changes in this PR. I think I adjusted everything the right way. However, I'm mostly unsure about the |
The description states options are optional but the way the method was currently typed they would be required. I've also adjusted the `package.json` slightly to define exports for all kinds of bundlers (e.g. the top-level `module` field was missing). AFAICT there are some issues with resolving stuff (at least based on my update PR). Relates to gnosisguild/zodiac-pilot#321
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
05862c3
to
a34f2e8
Compare
@@ -46,33 +47,33 @@ export function fromLegacyConnection( | |||
const delayModuleWaypoint = moduleType === KnownContracts.DELAY && { | |||
account: { | |||
type: AccountType.DELAY, | |||
prefixedAddress: modulePrefixedAddress, | |||
prefixedAddress: modulePrefixedAddress?.toLowerCase(), |
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.
@cristovaoth looking at this, it seems a bit awkward that ser-kit generally operates on lowercase prefixed addresses, but its formatPrefixedAddress
produces a checksummed version.
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.
code looks good
3f4c38c
to
7c6f91c
Compare
LFG 🚀 |
This PR contains the following updates:
^0.3.17
->^1.0.0
Release Notes
gnosisguild/ser-kit (ser-kit)
v1.0.0
Compare Source
⚠ BREAKING CHANGES
Features
Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.