Skip to content

Update license and icon based on @kyaa-dost's investigation #3777

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

Merged
merged 5 commits into from
Feb 24, 2021

Conversation

michael-hawker
Copy link
Member

Follow-on to replace #3765

Fixes the nuget license/icon file based on #3765, single commit.

@ghost
Copy link

ghost commented Feb 23, 2021

Thanks michael-hawker for opening a Pull Request! The reviewers will test the PR and highlight if there is any conflict or changes required. If the PR is approved we will proceed to merge the pull request 🙌

@michael-hawker michael-hawker added this to the 7.0 milestone Feb 23, 2021
@ghost ghost requested review from azchohfi, Kyaa-dost and Rosuavio February 23, 2021 17:25
@michael-hawker michael-hawker added build 🔥 next preview ✈️ Label for marking what we want to include in the next preview release for developers to try. nuget 📦 labels Feb 23, 2021
Based on @Kyaa-dost's initial investigation

Keeping old and new formats as not all clients support the new formats in all scenarios yet.
@michael-hawker
Copy link
Member Author

Looks good from NuGet Package Explorer:

image

Saw that VS doesn't support the license display properly yet, so thinking we should keep both properties. Going to try and add that back. Added comment here: NuGet/Home#8000 (comment)

Keep embedded license file though as well.
@michael-hawker
Copy link
Member Author

Hit #3775 issue again, merged main to see if we can get a good build again.

@michael-hawker
Copy link
Member Author

Alright, I think this gets us to be good enough with compliance:

image

@michael-hawker
Copy link
Member Author

@azchohfi you good with this?

@azchohfi
Copy link
Contributor

Ugh.... I don't mind, but its weird to embed the file and not use it.

@michael-hawker michael-hawker merged commit 2c88fd3 into CommunityToolkit:master Feb 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build 🔥 next preview ✈️ Label for marking what we want to include in the next preview release for developers to try. nuget 📦
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants