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

V1.0 rc.1 #133

Draft
wants to merge 257 commits into
base: main
Choose a base branch
from
Draft

V1.0 rc.1 #133

wants to merge 257 commits into from

Conversation

expede
Copy link
Member

@expede expede commented Aug 6, 2024

No description provided.

QuinnWilton and others added 30 commits September 27, 2023 17:02
Note that not all of them pass yet, and that the build tests are
impossible to support, as currently defined, due to JSON
canonicalization issues.

Fixtures from: ucan-wg/conformance-tests#13
This also generally cleans up the trait bounds, and removes
the unused Fact trait
Co-authored-by: Philipp Krüger <philipp.krueger1@gmail.com>
Co-authored-by: Philipp Krüger <philipp.krueger1@gmail.com>
… UCAN

Co-authored-by: Philipp Krüger <philipp.krueger1@gmail.com>
Co-authored-by: Philipp Krüger <philipp.krueger1@gmail.com>
Signed-off-by: Philipp Krüger <philipp.krueger1@gmail.com>
Also: Take `impl AsRef<str>` in some places instead of `String`
(and standardize on specifying it as an `impl`-in-place).
matheus23 and others added 29 commits March 22, 2024 00:54
* refactor: Make delegation::Store::get return Option

* refactor: Make delegation::Store::insert take &self instead of &mut self

* refactor: Make invocation::Store take &self instead of &mut self

* refactor: Make `delegation::Agent` not take `&mut self` in methods

* refactor: Make `Agent` take `DID` by value

* refactor: Take `DID` by value in `delegation::Agent::new`

* refactor: Change generic order in `delegation::Agent` and add defaults
…x-chain

Remove unrestricted powerbox chain
More bugfixes & some suggested changes
@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 24.18928% with 2291 lines in your changes missing coverage. Please review.

Project coverage is 13.54%. Comparing base (83528cc) to head (076ea4c).

Files Patch % Lines
src/delegation/policy/predicate.rs 8.79% 311 Missing and 114 partials ⚠️
src/ipld/promised.rs 0.00% 156 Missing ⚠️
src/did/key/verifier.rs 30.05% 77 Missing and 44 partials ⚠️
src/invocation/payload.rs 33.52% 31 Missing and 84 partials ⚠️
src/delegation/payload.rs 47.90% 20 Missing and 67 partials ⚠️
src/ability/msg/send.rs 14.77% 59 Missing and 16 partials ⚠️
src/ability/crud/update.rs 0.00% 71 Missing ⚠️
src/ability/crud/create.rs 0.00% 64 Missing ⚠️
src/ability/crud/read.rs 3.12% 59 Missing and 3 partials ⚠️
src/ability/crud.rs 9.37% 56 Missing and 2 partials ⚠️
... and 54 more

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

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##             main     #133       +/-   ##
===========================================
- Coverage   62.12%   13.54%   -48.58%     
===========================================
  Files          24       68       +44     
  Lines         895     5803     +4908     
  Branches      214     2762     +2548     
===========================================
+ Hits          556      786      +230     
- Misses        196     4428     +4232     
- Partials      143      589      +446     
Files Coverage Δ
src/ability/command.rs 100.00% <100.00%> (ø)
src/did.rs 0.00% <ø> (ø)
src/invocation/store/traits.rs 100.00% <100.00%> (ø)
src/reader.rs 3.75% <ø> (ø)
src/invocation.rs 93.33% <93.33%> (ø)
src/ipld.rs 13.79% <0.00%> (ø)
src/crypto/es512.rs 33.33% <33.33%> (ø)
src/delegation/store/traits.rs 60.00% <60.00%> (ø)
src/invocation/store/memory.rs 60.00% <60.00%> (ø)
src/ability/parse.rs 12.50% <12.50%> (ø)
... and 58 more

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