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

[master] Update dependencies from dotnet/arcade #277

Merged

Conversation

dotnet-maestro[bot]
Copy link
Contributor

@dotnet-maestro dotnet-maestro bot commented Mar 19, 2019

This pull request updates the following dependencies

Updates from https://github.com/dotnet/arcade

  • Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19167.10

From https://github.com/dotnet/arcade

  • Build: 20190317.10
  • Date Produced: 3/18/2019 7:14 AM
  • Commit: de7be3ba62b92e5c48c36876c851a14f154444af
  • Branch: refs/heads/master
  • Updates:
    • Microsoft.DotNet.Arcade.Sdk -> 1.0.0-beta.19167.10

From https://github.com/dotnet/arcade

  • Build: 20190320.2
  • Date Produced: 3/20/2019 5:43 PM
  • Commit: f70fa34786cff993625a4548ae0125335eabe82e
  • Branch: refs/heads/master
  • Updates:
    • Microsoft.DotNet.Arcade.Sdk -> 1.0.0-beta.19170.2

…317.10

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19167.10
@dotnet-maestro
Copy link
Contributor Author

dotnet-maestro bot commented Mar 19, 2019

Auto-Merge Status

This pull request has not been merged because Maestro++ is waiting on the following merge policies.

  • All Checks Successful Unsuccessful checks: public-CI

…320.2

- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19170.2
@jmarolf
Copy link
Contributor

jmarolf commented Mar 21, 2019

@tmat what is the correct way to fix this nuget licensing issue?

@dotnet-maestro
Copy link
Contributor Author

dotnet-maestro bot commented Mar 21, 2019

Auto-Merge Status

This pull request has been merged because the following merge policies have succeeded.

  • ✔️ All Checks Successful Succeeded - Successful checks: public-CI

@dotnet-maestro dotnet-maestro bot requested a review from a team as a code owner March 22, 2019 01:08
@@ -8,7 +8,12 @@
</PropertyGroup>

<Import Project="Sdk.props" Sdk="Microsoft.DotNet.Arcade.Sdk" />


<PropertyGroup Condition="'$(CopyrightMicrosoft)' != ''">
Copy link
Member

Choose a reason for hiding this comment

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

Condition="'$(CopyrightMicrosoft)' != ''" shouldn't be necessary anymore. Arcade now defines the property globally.

Copy link
Contributor

Choose a reason for hiding this comment

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

so just this?

  <PropertyGroup>
    <PackageLicenseExpression>Apache-2.0</PackageLicenseExpression>
  </PropertyGroup>

Copy link
Member

Choose a reason for hiding this comment

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

No, include the Copyright, just not the condition.

@dotnet-maestro dotnet-maestro bot merged commit 101b77e into master Mar 22, 2019
@dotnet-maestro dotnet-maestro bot deleted the darc-master-36a646e2-1925-4a5c-9726-c2fd3a992d89 branch March 22, 2019 01:28
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.

2 participants