Skip to content

Bump Microsoft.AspNetCore.Server.IIS from 2.2.0 to 2.2.6 in /src/Servers/IIS/IIS/test/testassets/InProcessNewShimWebSite #42624

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

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Jul 7, 2022

Bumps Microsoft.AspNetCore.Server.IIS from 2.2.0 to 2.2.6.

Release notes

Sourced from Microsoft.AspNetCore.Server.IIS's releases.

.NET Core 2.2.6

Repos

Commits
  • e7f262e Merge in 'release/2.2' changes
  • 2c9e5c9 Merge branch 'release/2.1' => 'release/2.2' #11236
  • d2327a5 Merge branch 'release/2.1' into release/2.2
  • 19dc9d1 Merge in 'release/2.2' changes
  • d858156 Fix order in which EF properties are set
  • 21f6b5f Update submodules.props for EF 2.2.6
  • f74ca43 Always include ANCM in build output (#11199)
  • 2b771cd Merge in 'release/2.2' changes
  • 0460bc3 Always include ANCM in build output (#11198)
  • 39c7419 Merge in 'release/2.2' changes
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
    You can disable automated security fix PRs for this repo from the Security Alerts page.

Bumps [Microsoft.AspNetCore.Server.IIS](https://github.com/aspnet/AspNetCore) from 2.2.0 to 2.2.6.
- [Release notes](https://github.com/aspnet/AspNetCore/releases)
- [Changelog](https://github.com/dotnet/aspnetcore/blob/main/docs/ReleasePlanning.md)
- [Commits](2.2.0...v2.2.6)

---
updated-dependencies:
- dependency-name: Microsoft.AspNetCore.Server.IIS
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot added .NET feature-blazor-native-dependencies This issue is related to WASM native embedded libraries labels Jul 7, 2022
@ghost ghost added the area-runtime label Jul 7, 2022
@Tratcher Tratcher requested a review from HaoK July 11, 2022 20:00
@dougbu dougbu enabled auto-merge (squash) July 11, 2022 20:05
@dougbu dougbu changed the title Bump Microsoft.AspNetCore.Server.IIS from 2.2.0 to 2.2.6 in /src/Servers/IIS/IIS/test/testassets/InProcessNewShimWebSite Remove Microsoft.AspNetCore.Server.IIS 2.2.0 from /src/Servers/IIS/IIS/test/testassets/InProcessNewShimWebSite Jul 11, 2022
@dougbu
Copy link
Contributor

dougbu commented Jul 11, 2022

Approved and enabled auto-squish

@dougbu
Copy link
Contributor

dougbu commented Jul 11, 2022

Ugh, the InProcessNewShimWebSite project only references 2.2-era packages and no new product code (aside from the installed ANCM shim). Should I change the test to use the current shared framework instead❔ Or, should the test asset and tests that use it be removed❔

@dougbu dougbu force-pushed the dependabot/nuget/src/Servers/IIS/IIS/test/testassets/InProcessNewShimWebSite/Microsoft.AspNetCore.Server.IIS-2.2.6 branch from 619a7d9 to ab518bb Compare July 12, 2022 04:41
@dougbu dougbu disabled auto-merge July 12, 2022 04:42
@dougbu dougbu changed the title Remove Microsoft.AspNetCore.Server.IIS 2.2.0 from /src/Servers/IIS/IIS/test/testassets/InProcessNewShimWebSite Bump Microsoft.AspNetCore.Server.IIS from 2.2.0 to 2.2.6 in /src/Servers/IIS/IIS/test/testassets/InProcessNewShimWebSite Jul 12, 2022
@dougbu
Copy link
Contributor

dougbu commented Jul 12, 2022

Simplest change was the original one. But we can still remove the test asset if anyone feels that's better.

Reverted the PR title too.

@dougbu
Copy link
Contributor

dougbu commented Jul 12, 2022

@Tratcher please approve if you agree on this approach

Copy link
Member

@Tratcher Tratcher left a comment

Choose a reason for hiding this comment

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

This is fine for now, but @HaoK, we should look at getting rid of such tests. They don't make sense anymore.

@dougbu dougbu merged commit 048a3fb into main Jul 12, 2022
@dougbu dougbu deleted the dependabot/nuget/src/Servers/IIS/IIS/test/testassets/InProcessNewShimWebSite/Microsoft.AspNetCore.Server.IIS-2.2.6 branch July 12, 2022 21:33
@ghost ghost added this to the 7.0-preview7 milestone Jul 12, 2022
@HaoK
Copy link
Member

HaoK commented Jul 19, 2022

@Tratcher exactly what tests are no longer needed, the new shim/old shim tests?

@Tratcher
Copy link
Member

The tests that rely on the 2.2 packages, unless we can refactor them to target 3.1.

@amcasey amcasey added area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions and removed area-runtime labels Jun 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions feature-blazor-native-dependencies This issue is related to WASM native embedded libraries
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants