search

temporalio / sdk-core

Core Temporal SDK that can be used as a base for language specific Temporal SDKs
MIT License
277 stars 77 forks source link

Add missing license to test-utils crate #657

Closed h7kanna closed 10 months ago

h7kanna commented 10 months ago

What was changed

Added license to test-utils crate to 'cargo-deny' failures

Why?

Checklist

  1. Closes

  2. How was this tested:

  3. Any docs updates needed?

cretz commented 10 months ago

Can we instead have all just refer to the license at the root?

h7kanna commented 10 months ago

Yes with license = { workspace = true }

cretz commented 10 months ago

@Sushisource - is it ok if we do that in each crate and remove all the LICENSE.txt files but the one in the root? Also, can we remove the unnecessary opening line on the root LICENSE.txt? It seems to be preventing GitHub from recognizing it's MIT for repo-metadata purposes.

Sushisource commented 10 months ago

Sure, we can use one shared one and get rid of the line at the top.

h7kanna commented 10 months ago

Ok, made the required changes