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

update nimbus #2267

Merged
merged 2 commits into from May 11, 2023
Merged

update nimbus #2267

merged 2 commits into from May 11, 2023

Conversation

librelois
Copy link
Collaborator

@librelois librelois commented Apr 26, 2023

What does it do?

Apply this nimbus PR on moonbeam: moonbeam-foundation/nimbus#89

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?

@librelois librelois added B5-clientnoteworthy Changes should be mentioned in any downstream projects' release notes D5-nicetohaveaudit⚠️ PR contains trivial changes to logic that should be properly reviewed. labels Apr 26, 2023
@librelois librelois added the not-breaking Does not need to be mentioned in breaking changes label Apr 26, 2023
@librelois librelois merged commit 504ac0e into master May 11, 2023
24 checks passed
@librelois librelois deleted the elois-nimbus-fix branch May 11, 2023 06:23
@notlesh notlesh added D1-audited👍 PR contains changes to fund-managing logic that has been properly reviewed and externally audited and removed D5-nicetohaveaudit⚠️ PR contains trivial changes to logic that should be properly reviewed. labels May 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
B5-clientnoteworthy Changes should be mentioned in any downstream projects' release notes D1-audited👍 PR contains changes to fund-managing logic that has been properly reviewed and externally audited not-breaking Does not need to be mentioned in breaking changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants