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

Lock file update #1997

Closed
wants to merge 1 commit into from
Closed

Conversation

girazoki
Copy link
Collaborator

What does it do?

Fixes dalek host function to take signature as a reference instead of as an slice, which triggers a panick when creating the signature

What important points reviewers should know?

Is there something left for follow-up PRs?

What alternative implementations were considered?

Are there relevant PRs or issues in other repositories (Substrate, Polkadot, Frontier, Cumulus)?

What value does it bring to the blockchain users?

@girazoki girazoki added B5-clientnoteworthy Changes should be mentioned in any downstream projects' release notes B0-silent Changes should not be mentioned in any release notes labels Dec 12, 2022
@girazoki
Copy link
Collaborator Author

Closed in favour of #1998

@girazoki girazoki closed this Dec 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
B0-silent Changes should not be mentioned in any release notes B5-clientnoteworthy Changes should be mentioned in any downstream projects' release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants