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.

Sign up for GitHub

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

Jump to bottom

Deploy to mainnet #1632

Merged
merged 11 commits into from
Feb 21, 2024
Merged

Deploy to mainnet #1632

merged 11 commits into from
Feb 21, 2024

Conversation

github-actions[bot]
Copy link
Contributor

No description provided.

zmcNotafraid and others added 9 commits February 6, 2024 12:40
@zmcNotafraid
Issue 424 3 (#1622)
11621ff 
* feat: remove ckb_hodl_waves column from statistic_infos

Signed-off-by: Miles Zhang <[email protected]>

* feat: add ckb_hodl_waves to daily_statistic

Signed-off-by: Miles Zhang <[email protected]>

* feat: calculate ckb_hodl_wave and save to daily_statistics

Signed-off-by: Miles Zhang <[email protected]>

* fix: fix typo

Signed-off-by: Miles Zhang <[email protected]>

---------

Signed-off-by: Miles Zhang <[email protected]>
@zmcNotafraid
feat: add missing attribute for cell output (#1624)
8bb8267 
Signed-off-by: Miles Zhang <[email protected]>
@zmcNotafraid
feat: use task to reset ckb_hodl_wave (#1626)
b076be9 
Signed-off-by: Miles Zhang <[email protected]>
@rabbitz
fix: adjust the count of nft cells change (#1625)
0fbdb0d
@rabbitz
fix: cell input creation for pending transactions sync (#1628)
f65e75b
@zmcNotafraid
fix: ckb hodl wave needs to calcualte dead cell_output after counted … (
78e3feb 
#1629)

fix: ckb hodl wave needs to calcualte dead cell_output after counted date

Signed-off-by: Miles Zhang <[email protected]>
@rabbitz
fix: cal pending output balance occupied (#1630)
6a85a00
@zmcNotafraid
fix: ckb_hodl_wave's calcualte needs to use dao's c_i attribute (#1631)
0f9ef08 
Signed-off-by: Miles Zhang <[email protected]>
@rabbitz
Merge pull request #1623 from nervosnetwork/develop
e3cc632 
Deploy to testnet
@codecov-commenter
Copy link

Codecov Report

Attention: 37 lines in your changes are missing coverage. Please review.

Comparison is base (6ac53e0) 67.18% compared to head (ca9f48d) 66.25%.
Report is 19 commits behind head on master.

❗ Current head ca9f48d differs from pull request most recent head d77219e. Consider uploading reports for the commit d77219e to get more accurate results

Files Patch % Lines
app/controllers/concerns/cell_data_comparator.rb 41.66% 21 Missing ⚠️
app/models/daily_statistic.rb 80.00% 6 Missing ⚠️
app/serializers/cell_output_serializer.rb 44.44% 5 Missing ⚠️
...sks/migration/reset_daily_statistic_attribute.rake 37.50% 5 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##           master    #1632      +/-   ##
==========================================
- Coverage   67.18%   66.25%   -0.93%     
==========================================
  Files         290      310      +20     
  Lines        7969     8488     +519     
==========================================
+ Hits         5354     5624     +270     
- Misses       2615     2864     +249

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@rabbitz rabbitz added this pull request to the merge queue Feb 21, 2024
Merged via the queue into master with commit 8b50d25 Feb 21, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zmcNotafraid zmcNotafraid zmcNotafraid approved these changes

@rabbitz rabbitz rabbitz approved these changes

@Keith-CY Keith-CY Awaiting requested review from Keith-CY

Assignees
No one assigned
Labels
auto-pr
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@codecov-commenter @rabbitz @zmcNotafraid @Keith-CY