search

aws / aws-nitro-enclaves-cli

Tooling for Nitro Enclave Management
Apache License 2.0
124 stars 81 forks source link

Update THIRD_PARTY_LICENSES_RUST_CRATES.html #427

Closed raulmldv closed 1 year ago

raulmldv commented 1 year ago

Fixed failing nitro-about target on recent PRs.

Signed-off-by: Raul-Ovidiu Moldovan raulmldv@amazon.com

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

petreeftime commented 1 year ago

I think we should simply remove nitro-about from the default test set, or keep just a check for valid licenses. We only need to include this file when we produce a binary distribution.

raulmldv commented 1 year ago

I think we should simply remove nitro-about from the default test set, or keep just a check for valid licenses. We only need to include this file when we produce a binary distribution.

Couldn't find a quick check using cargo-about or cargo-license. I will remove nitro-about for now and look more into it. I think I can leave the licenses updated so I'll push here.