Skip to content

Address some of the comments that were made on the PR to dmf#12

Merged
pac-work merged 15 commits intopac-work:rust-bindingsfrom
bisect-pt:pf/pr-comments
Oct 8, 2025
Merged

Address some of the comments that were made on the PR to dmf#12
pac-work merged 15 commits intopac-work:rust-bindingsfrom
bisect-pt:pf/pr-comments

Conversation

@pedro-alves-ferreira
Copy link

No description provided.

cargo audit was being installed twice

Signed-off-by: Pedro Ferreira <pedro@bisect.pt>
Signed-off-by: Pedro Ferreira <pedro@bisect.pt>
Signed-off-by: Pedro Ferreira <pedro@bisect.pt>
Signed-off-by: Pedro Ferreira <pedro@bisect.pt>
cargo audit complained about RUSTSEC-2025-0055

Signed-off-by: Pedro Ferreira <pedro@bisect.pt>
Signed-off-by: Pedro Ferreira <pedro@bisect.pt>
Asked on Slack for guidance regarding which licenses should be accepted.

Signed-off-by: Pedro Ferreira <pedro@bisect.pt>
Signed-off-by: Pedro Ferreira <pedro@bisect.pt>
Signed-off-by: Pedro Ferreira <pedro@bisect.pt>
Signed-off-by: Pedro Ferreira <pedro@bisect.pt>
@pac-work
Copy link
Owner

pac-work commented Oct 8, 2025

Some of the commits seems to not be signed off, @pedro-alves-ferreira .

The api is now returned as an Arc, which
can be cloned.

Signed-off-by: Pedro Ferreira <pedro@bisect.pt>
Signed-off-by: Pedro Ferreira <pedro@bisect.pt>
Signed-off-by: Pedro Ferreira <pedro@bisect.pt>
Signed-off-by: Pedro Ferreira <pedro@bisect.pt>
Signed-off-by: Pedro Ferreira <pedro@bisect.pt>
@pac-work pac-work merged commit 0a73ed2 into pac-work:rust-bindings Oct 8, 2025
0 of 4 checks passed
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