Skip to content

[release/3.1] Correct baseline checks #25217

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 1 commit into from
Sep 9, 2020
Merged

Conversation

dougbu
Copy link
Contributor

@dougbu dougbu commented Aug 24, 2020

  • disable use of @(SuppressBaselineReference) items while servicing
  • update BaselineGenerator to produce baselines useful in 5.0
  • update Baseline.Designer.props using new generator

nit: do not generate empty <ItemGroup /> elements

@dougbu dougbu requested a review from a team August 24, 2020 22:32
@ghost ghost added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Aug 24, 2020
@dougbu dougbu added the tell-mode Indicates a PR which is being merged during tell-mode label Aug 24, 2020
@dougbu dougbu force-pushed the dougbu/fixup.baselines branch from 83c2ace to 26cd396 Compare August 24, 2020 22:34
@dougbu dougbu added this to the 3.1.9 milestone Aug 25, 2020
@dougbu dougbu added the blocked The work on this issue is blocked due to some dependency label Aug 25, 2020
@dougbu
Copy link
Contributor Author

dougbu commented Aug 25, 2020

Blocked until branch opens for 3.1.9. But, please review soon-ish so we're ready😺

dougbu added a commit that referenced this pull request Aug 25, 2020
- mostly duplicates #25217
- update `BaselineGenerator` to produce baselines useful in 6.0 (too)
- update Baseline.Designer.props using new generator (matching 3.1.7 release)
- always suppress references expressed only in `*.nuspec` files
  - needed even in servicing builds
- restore warning and errors about removed references (new for 5.0)
  - adjust exclusions to handle `@(_ProjectReferenceByAssemblyName)` removal

nit: do not generate empty `<ItemGroup />` elements
dougbu added a commit that referenced this pull request Aug 25, 2020
- mostly duplicates #25217
- update `BaselineGenerator` to produce baselines useful in 6.0 (too)
- update Baseline.Designer.props using new generator (matching 3.1.7 release)
- always suppress references expressed only in `*.nuspec` files
  - needed even in servicing builds
- restore warning and errors about removed references (new for 5.0)
  - adjust exclusions to handle `@(_ProjectReferenceByAssemblyName)` removal

nit: do not generate empty `<ItemGroup />` elements
dougbu added a commit that referenced this pull request Aug 28, 2020
- mostly duplicates #25217
- update `BaselineGenerator` to produce baselines useful in 6.0 (too)
- update Baseline.Designer.props using new generator (matching 3.1.7 release)
- always suppress references expressed only in `*.nuspec` files
  - needed even in servicing builds
- restore warning and errors about removed references (new for 5.0)
  - adjust exclusions to handle `@(_ProjectReferenceByAssemblyName)` removal

nit: do not generate empty `<ItemGroup />` elements
dougbu added a commit that referenced this pull request Aug 28, 2020
- mostly duplicates #25217
- update `BaselineGenerator` to produce baselines useful in 6.0 (too)
- update Baseline.Designer.props using new generator (matching 3.1.7 release)
- always suppress references expressed only in `*.nuspec` files
  - needed even in servicing builds
- restore warning and errors about removed references (new for 5.0)
  - adjust exclusions to handle `@(_ProjectReferenceByAssemblyName)` removal

nit: do not generate empty `<ItemGroup />` elements

* Correct `@(SuppressBaselineReference)` items
  - remove out-of-date `@(SuppressBaselineReference)` items
    - either 3.1.7 baselines we're using don't include reference or still using package
    - fix some comments and `Condition` attributes to make remainder easy to find
  - add missing `@(SuppressBaselineReference)` items
@wtgodbe
Copy link
Member

wtgodbe commented Sep 8, 2020

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

- disable use of `@(SuppressBaselineReference)` items while servicing
- update `BaselineGenerator` to produce baselines useful in 5.0
- update Baseline.Designer.props using new generator

nit: do not generate empty `<ItemGroup />` elements
@dougbu dougbu force-pushed the dougbu/fixup.baselines branch from abe386e to 09a6539 Compare September 9, 2020 02:41
@dougbu dougbu removed the blocked The work on this issue is blocked due to some dependency label Sep 9, 2020
@dougbu dougbu merged commit 473bdc0 into release/3.1 Sep 9, 2020
@dougbu dougbu deleted the dougbu/fixup.baselines branch September 9, 2020 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework tell-mode Indicates a PR which is being merged during tell-mode
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants