Skip to content

Comments

Log an error if imported in Directory.Build.props#322

Merged
jeffkl merged 1 commit intomicrosoft:mainfrom
jeffkl:error-if-imported-in-directory-build-props
Nov 25, 2021
Merged

Log an error if imported in Directory.Build.props#322
jeffkl merged 1 commit intomicrosoft:mainfrom
jeffkl:error-if-imported-in-directory-build-props

Conversation

@jeffkl
Copy link
Contributor

@jeffkl jeffkl commented Nov 25, 2021

Since the MSBuild Update gesture is used, Microsoft.Build.CentralPackageVersions must be included in Directory.Build.targets not Directory.Build.props. This change logs an error so that users will know when they've made this mistake.

Fixes #179

Since the MSBuild Update gesture is used, Microsoft.Build.CentralPackageVersions must be included in Directory.Build.targets not Directory.Build.props.  This change logs an error so that users will know when they've made this mistake.

Fixes #179
@jeffkl jeffkl merged commit af63eab into microsoft:main Nov 25, 2021
@jeffkl jeffkl deleted the error-if-imported-in-directory-build-props branch November 25, 2021 21:00
jeffkl added a commit that referenced this pull request Dec 2, 2021
…om import pattern. (#325)

#322 didn't take into account that if Microsoft.Build.CentralPackageVersions is imported in a custom way.  This change now makes sure it was imported after Microsoft.Common.targets

Fixes #324
jeffkl added a commit that referenced this pull request Dec 3, 2021
…g a custom import pattern" (#327)

* Revert "Fix issue with CentralPackageVersions logging error when using a custom import pattern. (#325)"

This reverts commit 99a3dc3.

* Revert "Log an error if imported in Directory.Build.props (#322)"

This reverts commit af63eab.

Reopens #324 and #179
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.

MicrosoftCommonPropsHasBeenImported is set after importing Directory.Build.props

1 participant