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

Fix lints from aries_vcx_core #988

Merged
merged 2 commits into from
Sep 21, 2023

Conversation

bobozaur
Copy link
Contributor

PR that addresses lints from aries_vcx_core crate.

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>
@codecov-commenter
Copy link

codecov-commenter commented Sep 20, 2023

Codecov Report

Merging #988 (e9eb1b0) into fix/lints_and_format (8c3fb06) will decrease coverage by 0.01%.
The diff coverage is 27.02%.

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

@@                   Coverage Diff                    @@
##           fix/lints_and_format     #988      +/-   ##
========================================================
- Coverage                 30.05%   30.05%   -0.01%     
========================================================
  Files                       417      415       -2     
  Lines                     26967    26937      -30     
  Branches                   5229     5227       -2     
========================================================
- Hits                       8105     8095      -10     
+ Misses                    16662    16648      -14     
+ Partials                   2200     2194       -6     
Flag Coverage Δ
unittests-aries-vcx 30.05% <27.02%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
aries_vcx_core/src/anoncreds/indy/proofs/prover.rs 0.00% <0.00%> (ø)
...ies_vcx_core/src/anoncreds/indy/proofs/verifier.rs 0.00% <ø> (ø)
aries_vcx_core/src/anoncreds/indy_anoncreds.rs 0.00% <0.00%> (ø)
...ries_vcx_core/src/errors/mapping_indy_api_types.rs 20.58% <ø> (ø)
aries_vcx_core/src/ledger/common.rs 0.00% <ø> (ø)
aries_vcx_core/src/wallet/indy/indy_wallet.rs 54.25% <ø> (ø)
aries_vcx_core/src/wallet/indy/mod.rs 46.55% <0.00%> (ø)
aries_vcx_core/src/wallet/indy/signing.rs 42.00% <ø> (ø)
aries_vcx_core/src/wallet/indy/wallet.rs 45.39% <ø> (ø)
libvdrtools/src/controllers/anoncreds/prover.rs 1.08% <ø> (ø)
... and 4 more

... and 3 files with indirect coverage changes

@bobozaur bobozaur changed the title Fixes lints from aries_vcx_core Fix lints from aries_vcx_core Sep 20, 2023
@bobozaur bobozaur changed the base branch from main to fix/lints_and_format September 20, 2023 12:15
Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>
@Patrik-Stas Patrik-Stas merged commit 4b6c15b into fix/lints_and_format Sep 21, 2023
@Patrik-Stas Patrik-Stas deleted the fix/aries_vcx_core_lints branch September 21, 2023 08:17
bobozaur added a commit that referenced this pull request Sep 21, 2023
* Fix lints from aries_vcx_core (#988)

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>
mirgee pushed a commit that referenced this pull request Sep 21, 2023
* Updated rustfmt.toml

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Added lints section in workspace

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed napi-rs wrapper lints

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed uniffi_aries_vcx lints

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed tools crate lints

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed public_key crate lints

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed indy_ledger_response_parser crate lints

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed did_resolver_web lints

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed did_resolver_sov crate lints

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed did_resolver_registry crate lints

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed did_peer crate lints

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed did_parser crate lints

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed did_doc_sov crate lints

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed did_doc crate lints

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed rust agent lints

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed agency_client crate lints

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Uniformized toolchain version in CI

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Ran cargo fmt

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Revert Rust toolchain version

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fix code formatting CI check

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* More formatting fixes

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Macro formatting fix

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed ResponseParser construction

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fixed rust agent lints post rebase

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fix lints from aries_vcx_core (#988)

* Fix lints from aries_vcx_core (#988)

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

* Fix lints in aries_vcx (#989)

Addressed lints and formatting in aries_vcx.

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>

---------

Signed-off-by: Bogdan Mircea <mirceapetrebogdan@gmail.com>
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.

4 participants