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

feat(script): remove useless scripts #5580

Merged
merged 1 commit into from
Nov 13, 2023

Conversation

forfreeday
Copy link
Contributor

What does this PR do?
Delete shell scripts that are no longer used.

Why are these changes required?
These scripts are no longer used and do not need to exist in the project.

This PR has been tested by:

  • Unit Tests
  • Manual Testing

Follow up

Extra details

@codecov-commenter
Copy link

Codecov Report

Merging #5580 (e7daf14) into develop (de23b55) will increase coverage by 1.76%.
Report is 53 commits behind head on develop.
The diff coverage is 70.00%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@              Coverage Diff              @@
##             develop    #5580      +/-   ##
=============================================
+ Coverage      64.64%   66.40%   +1.76%     
- Complexity      9689    10212     +523     
=============================================
  Files            867      894      +27     
  Lines          51724    53841    +2117     
  Branches        5754     5931     +177     
=============================================
+ Hits           33435    35755    +2320     
+ Misses         15688    15295     -393     
- Partials        2601     2791     +190     
Files Coverage Δ
...g/tron/core/actuator/DelegateResourceActuator.java 89.03% <100.00%> (+0.64%) ⬆️
...c/main/java/org/tron/core/db/TransactionCache.java 100.00% <100.00%> (ø)
...ava/org/tron/common/parameter/CommonParameter.java 59.82% <100.00%> (+0.05%) ⬆️
common/src/main/java/org/tron/core/Constant.java 50.00% <ø> (ø)
...mon/logsfilter/nativequeue/NativeMessageQueue.java 78.37% <100.00%> (+1.23%) ⬆️
framework/src/main/java/org/tron/core/Wallet.java 58.64% <ø> (+9.91%) ⬆️
.../main/java/org/tron/core/config/DefaultConfig.java 100.00% <ø> (ø)
...mework/src/main/java/org/tron/core/db/Manager.java 57.69% <ø> (+0.44%) ⬆️
...c/main/java/org/tron/core/net/TronNetDelegate.java 46.92% <100.00%> (+0.82%) ⬆️
...in/java/org/tron/core/net/peer/PeerConnection.java 89.20% <ø> (+19.20%) ⬆️
... and 20 more

... and 87 files with indirect coverage changes

📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

Copy link
Contributor

@halibobo1205 halibobo1205 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@halibobo1205 halibobo1205 merged commit df52667 into tronprotocol:develop Nov 13, 2023
6 checks passed
@halibobo1205 halibobo1205 added this to the GreatVoyage-v4.7.4 milestone Nov 13, 2023
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.

None yet

3 participants