Skip to content

feat: backport release/v1.6.x to main#1926

Merged
thomas-nguy merged 5 commits into
crypto-org-chain:mainfrom
thomas-nguy:thomas/rebase-1.6.x
Dec 2, 2025
Merged

feat: backport release/v1.6.x to main#1926
thomas-nguy merged 5 commits into
crypto-org-chain:mainfrom
thomas-nguy:thomas/rebase-1.6.x

Conversation

@thomas-nguy

Copy link
Copy Markdown
Collaborator

👮🏻👮🏻👮🏻 !!!! REFERENCE THE PROBLEM YOUR ARE SOLVING IN THE PR TITLE AND DESCRIBE YOUR SOLUTION HERE !!!! DO NOT FORGET !!!! 👮🏻👮🏻👮🏻

PR Checklist:

  • Have you read the CONTRIBUTING.md?
  • Does your PR follow the C4 patch requirements?
  • Have you rebased your work on top of the latest master?
  • Have you checked your code compiles? (make)
  • Have you included tests for any non-trivial functionality?
  • Have you checked your code passes the unit tests? (make test)
  • Have you checked your code formatting is correct? (go fmt)
  • Have you checked your basic code style is fine? (golangci-lint run)
  • If you added any dependencies, have you checked they do not contain any known vulnerabilities? (go list -json -m all | nancy sleuth)
  • If your changes affect the client infrastructure, have you run the integration test?
  • If your changes affect public APIs, does your PR follow the C4 evolution of public contracts?
  • If your code changes public APIs, have you incremented the crate version numbers and documented your changes in the CHANGELOG.md?
  • If you are contributing for the first time, please read the agreement in CONTRIBUTING.md now and add a comment to this pull request stating that your PR is in accordance with the Developer's Certificate of Origin.

Thank you for your code, it's appreciated! :)

* check authorisation list in e2ee

* fix golang-ci

* fix upgrade

* add integration tests
* add check destination address

* add changelog

* add nil check
…in#1924)

* add missing authorisation address

* add changelog

* remove extra line
@thomas-nguy
thomas-nguy requested a review from a team as a code owner December 2, 2025 05:42
@thomas-nguy
thomas-nguy requested review from XinyuCRO and calvinaco and removed request for a team December 2, 2025 05:42
@coderabbitai

coderabbitai Bot commented Dec 2, 2025

Copy link
Copy Markdown
Contributor

Warning

Rate limit exceeded

@thomas-nguy has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 22 minutes and 15 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 9be49c0 and c181b9e.

📒 Files selected for processing (5)
  • CHANGELOG.md (1 hunks)
  • app/block_address.go (2 hunks)
  • app/proposal.go (2 hunks)
  • integration_tests/test_e2ee.py (2 hunks)
  • integration_tests/test_mempool.py (1 hunks)
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@thomas-nguy thomas-nguy changed the title feat: backport release v1.6.x to main feat: backport release/v1.6.x to main Dec 2, 2025
@codecov

codecov Bot commented Dec 2, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 16 lines in your changes missing coverage. Please review.
✅ Project coverage is 37.13%. Comparing base (a3c2f70) to head (c181b9e).
⚠️ Report is 123 commits behind head on main.

Files with missing lines Patch % Lines
app/proposal.go 0.00% 11 Missing ⚠️
app/block_address.go 0.00% 5 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##             main    #1926       +/-   ##
===========================================
+ Coverage   16.87%   37.13%   +20.25%     
===========================================
  Files          72      127       +55     
  Lines        6163     9757     +3594     
===========================================
+ Hits         1040     3623     +2583     
- Misses       5000     5713      +713     
- Partials      123      421      +298     
Files with missing lines Coverage Δ
app/block_address.go 11.76% <0.00%> (-9.98%) ⬇️
app/proposal.go 28.57% <0.00%> (+10.15%) ⬆️

... and 119 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@thomas-nguy
thomas-nguy added this pull request to the merge queue Dec 2, 2025
Merged via the queue into crypto-org-chain:main with commit 01cafee Dec 2, 2025
44 of 45 checks passed
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.

2 participants