Skip to content
This repository has been archived by the owner on Dec 5, 2024. It is now read-only.

Allow for multiple license files #342

Merged
merged 6 commits into from
May 4, 2023
Merged

Allow for multiple license files #342

merged 6 commits into from
May 4, 2023

Conversation

dhirschfeld
Copy link
Contributor

Resolves #341

@dhirschfeld dhirschfeld marked this pull request as draft May 1, 2023 11:53
@dhirschfeld dhirschfeld closed this May 1, 2023
@dhirschfeld dhirschfeld reopened this May 1, 2023
@dhirschfeld
Copy link
Contributor Author

Getting late here so I'll have to leave this for now. Feel free to pick this up / push any changes if you want to get it in quickly. Otherwise, I'll try and pick this up again tomorrow evening...

@wolfv
Copy link
Member

wolfv commented May 1, 2023

Looks good to me! :)

@dhirschfeld dhirschfeld marked this pull request as ready for review May 1, 2023 12:27
@wolfv
Copy link
Member

wolfv commented May 4, 2023

Looks like we need some fake licenses in that folder :)

@dhirschfeld
Copy link
Contributor Author

Looks like we need some fake licenses in that folder :)

Yup, will hopefully get some time tonight to finish this off...

@dhirschfeld
Copy link
Contributor Author 

ValueError: License file given in about/license_file (D:\a\boa\boa\tests\recipes\multiple_license\LICENSE) does not exist in source root dir or in recipe root dir (with meta.yaml)

@wolfv wolfv merged commit c72e361 into mamba-org:main May 4, 2023
@wolfv
Copy link
Member

wolfv commented May 4, 2023

thanks!

@dhirschfeld dhirschfeld deleted the multiple-license-files branch May 4, 2023 23:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Allow for more than one license, including directories
2 participants
@dhirschfeld @wolfv