Skip to content

Conversation

randy-cro
Copy link

@randy-cro randy-cro commented Sep 25, 2025

Description

Details in crypto-org-chain/cosmos-sdk#1725

Closes: #XXX

Description


For contributor use:

  • Targeted PR against correct branch (see CONTRIBUTING.md)
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
  • Code follows the module structure standards.
  • Wrote unit and integration tests
  • Updated relevant documentation (docs/) or specification (x/<module>/spec/)
  • Added relevant godoc comments.
  • Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
  • Re-reviewed Files changed in the Github PR explorer

For admin use:

  • Added appropriate labels to PR (ex. WIP, R4R, docs, etc)
  • Reviewers assigned
  • Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)

yihuang and others added 10 commits April 8, 2025 07:09
…hain#577) (crypto-org-chain#578)

* Problem: eth_getLogs miss logs with batch transactions. (crypto-org-chain#577)

* Problem: eth_getLogs miss logs with batch transactions.

* Update CHANGELOG.md

Signed-off-by: yihuang <[email protected]>

---------

Signed-off-by: yihuang <[email protected]>

* fix ci

---------

Signed-off-by: yihuang <[email protected]>
Co-authored-by: mmsqe <[email protected]>
…n#574 crypto-org-chain#576) (crypto-org-chain#579)

* Problem: dependency contains personal fork (crypto-org-chain#571)

* Problem: incorrect spendable balance when debug trace tx (crypto-org-chain#574)

* Problem: incorrect spendable balance when debug trace tx

* fix

* Update tests/integration_tests/test_tracers.py

Co-authored-by: mmsqe <[email protected]>
Signed-off-by: yihuang <[email protected]>

---------

Signed-off-by: yihuang <[email protected]>
Co-authored-by: yihuang <[email protected]>

* Problem: path parameter in debug_goTrace is not safe when serve public rpc (crypto-org-chain#576)

* Problem: no specific dir for the log from debug_goTrace

* lint

* allow any

* revert

* cleanup

* fix format

* fix config

* Apply suggestions from code review

---------

Co-authored-by: HuangYi <[email protected]>

---------

Signed-off-by: yihuang <[email protected]>
Co-authored-by: yihuang <[email protected]>
* add support for eth_getblockreceipts

* update changelog

* Update rpc/backend/blocks.go

Co-authored-by: Xinyu <[email protected]>
Signed-off-by: Thomas Nguy <[email protected]>

* fix lint

* optimization

* remove empty space

* add test case

* fix length and add test

* return err details

* fix tests

* fix lint

* fix tests

---------

Signed-off-by: Thomas Nguy <[email protected]>
Co-authored-by: Xinyu <[email protected]>
* add max gas cap in eth_call

* move logic
@randy-cro randy-cro force-pushed the test/optimise-endblocker-staking branch from 9ae38ca to e350514 Compare September 25, 2025 09:12
@randy-cro randy-cro changed the title Test/optimise endblocker staking Perf: optimise staking endblocker Sep 25, 2025
Copy link

codecov bot commented Sep 25, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 46.21%. Comparing base (86f043d) to head (386d3fc).
⚠️ Report is 1 commits behind head on release/v0.22.x.

Additional details and impacted files

Impacted file tree graph

@@               Coverage Diff                @@
##           release/v0.22.x     #743   +/-   ##
================================================
  Coverage            46.21%   46.21%           
================================================
  Files                  187      187           
  Lines                18938    18940    +2     
================================================
+ Hits                  8752     8754    +2     
  Misses                9399     9399           
  Partials               787      787           
Files with missing lines Coverage Δ
evmd/app.go 79.44% <100.00%> (+0.05%) ⬆️
🚀 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.

@randy-cro randy-cro changed the title Perf: optimise staking endblocker perf: optimise staking endblocker Sep 26, 2025
@thomas-nguy
Copy link

I think you can add your fix to the branch release/v0.22.x and release/v1.5.x for cronos because we just upgraded testnet to v1.5

@randy-cro
Copy link
Author

randy-cro commented Sep 29, 2025

I think you can add your fix to the branch release/v0.22.x and release/v1.5.x for cronos because we just upgraded testnet to v1.5

oh alright will backport these changes to those branches too

@thomas-nguy
Copy link

maybe change the PR target to release/v0.22.x or re-open the PR?

@randy-cro randy-cro force-pushed the test/optimise-endblocker-staking branch from 25a3554 to 01fc9a4 Compare September 30, 2025 04:13
@randy-cro randy-cro changed the base branch from release/v1.4.x-2 to release/v0.22.x September 30, 2025 04:25
@randy-cro randy-cro requested a review from a team as a code owner September 30, 2025 04:25
@randy-cro randy-cro requested review from calvinaco and removed request for a team September 30, 2025 04:25
@randy-cro randy-cro requested a review from songgaoye September 30, 2025 04:25

This comment has been minimized.

Copy link

socket-security bot commented Oct 6, 2025

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatedgolang/​github.com/​crypto-org-chain/​cosmos-sdk@​v0.50.6-0.20250424063720-28ea58ae20d8 ⏵ v0.50.6-0.20251015030247-d8ed3ff5806b76 +1100100100100

View full report

@randy-cro randy-cro force-pushed the test/optimise-endblocker-staking branch from 363c608 to 36818c8 Compare October 8, 2025 10:21
@randy-cro randy-cro merged commit c8a2716 into crypto-org-chain:release/v0.22.x Oct 15, 2025
33 of 35 checks passed
@randy-cro randy-cro deleted the test/optimise-endblocker-staking branch October 15, 2025 03:56
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.

3 participants