Skip to content
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

Added thirt party notice to nuget packages using third party package #249

Merged
merged 4 commits into from
Nov 25, 2016
Merged

Added thirt party notice to nuget packages using third party package #249

merged 4 commits into from
Nov 25, 2016

Conversation

Faizan2304
Copy link
Contributor

No description provided.

Copy-Item -Recurse $tpSrcDir\package\"_._" $stagingDir -Force
# Copy over empty and third patry notice file
Copy-Item $tpSrcDir\package\"_._" $stagingDir -Force
Copy-Item $tpSrcDir\package\"ThirdPartyNotices.txt" $stagingDir -Force
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: indent

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removing -Recurse is ok?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes. we are copying one file only

@@ -6,11 +6,12 @@
<title>Microsoft.TestPlatform.ObjectModel</title>
<authors>Microsoft</authors>
<owners>Microsoft</owners>
<requireLicenseAcceptance>true</requireLicenseAcceptance>
<requireLicenseAcceptance>false</requireLicenseAcceptance>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should require license acceptance?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed!!

@codito
Copy link
Contributor

codito commented Nov 25, 2016

👍

@Faizan2304 Faizan2304 merged commit 39eeb32 into microsoft:master Nov 25, 2016
@Faizan2304 Faizan2304 deleted the thirtPartyNotice branch November 25, 2016 12:13
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.

5 participants