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

Add tars sdk impl #818

Merged
merged 15 commits into from
Aug 28, 2023
Merged

Add tars sdk impl #818

merged 15 commits into from
Aug 28, 2023

Conversation

morebtcg
Copy link
Member

Add tars sdk impl

@sonarcloud
Copy link

sonarcloud bot commented Aug 28, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information


sendTransaction.setCallback(
new Callback() {
public void onMessage() {

Check notice

Code scanning / CodeQL

Missing Override annotation Note

This method overrides
Callback.onMessage
; it is advisable to add an Override annotation.
@codecov
Copy link

codecov bot commented Aug 28, 2023

Codecov Report

❗ No coverage uploaded for pull request base (release-3.5.0@cf1cfff). Click here to learn what that means.
Patch has no changes to coverable lines.

❗ Current head 929ab7b differs from pull request most recent head 850e7d6. Consider uploading reports for the commit 850e7d6 to get more accurate results

Additional details and impacted files
@@               Coverage Diff                @@
##             release-3.5.0     #818   +/-   ##
================================================
  Coverage                 ?   40.33%           
  Complexity               ?     2022           
================================================
  Files                    ?      386           
  Lines                    ?    14933           
  Branches                 ?     1719           
================================================
  Hits                     ?     6023           
  Misses                   ?     8374           
  Partials                 ?      536           

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

@morebtcg morebtcg merged commit d107ca6 into FISCO-BCOS:release-3.5.0 Aug 28, 2023
8 of 9 checks passed
@morebtcg morebtcg deleted the release-3.5.0 branch August 28, 2023 09:39
kyonRay pushed a commit to kyonRay/java-sdk that referenced this pull request Oct 12, 2023
kyonRay added a commit that referenced this pull request Oct 12, 2023
* <feat>(rpc): add call with sign interface of rpc. (#771)

* <feat>(rpc): add call with sign interface of rpc.

* <test>(integration): add v3.3.0 node test.

* <fix>(CI): fix integration test of extra data and input. (#796)

* <fix>(codec): fix int type not count bit size bug. (#800)

* upgrade openjdk version

* <fix>(test): remove unnecessary dependency com.google.guava. (#811)

* Add tars sdk impl (#818)

Add tars sdk impl (#818)

* <feat&fix>(transaction): fix call with sign result not handle revert message bug, add system features. (#819)

* Update tars client impl (#821)

* Add TarsClient

* Update bulid.gradle, add tars-sdk

* Update fisco-bcos-tars-sdk package name

* Using google code style

* Merge origin

* Revert changes of xml

* ExtraData use string type

* Update tars client

* Add tars config and impl

* Update loadLibrary

* Use google java format

* Update library path find

* Update tars client

---------

Co-authored-by: More <More@MSI>
Co-authored-by: More <[email protected]>

* <feat>(client): add getNodeListByType interface. (#822)

* Fix tars bug (#823)

* Add TarsClient

* Update bulid.gradle, add tars-sdk

* Update fisco-bcos-tars-sdk package name

* <fix>(client): add default nodeName to send request. (#825)

* Support jni inside jar (#826)

* Update sdk usage

* Support jni inside jar

---------

Co-authored-by: wenlinli <[email protected]>
Co-authored-by: MO NAN <[email protected]>
Co-authored-by: More <More@MSI>
Co-authored-by: More <[email protected]>
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