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

Bump OctoKit to 10.0.0 #3333

Merged
merged 8 commits into from
Feb 23, 2024
Merged

Conversation

dotnet-maestro[bot]
Copy link
Contributor

@dotnet-maestro dotnet-maestro bot commented Feb 23, 2024

dotnet/dnceng#2065

From https://github.com/dotnet/dnceng-shared

  • Subscription: 2873955e-d663-4fe5-b2be-08db410cf6aa
  • Build: 20240222.2
  • Date Produced: February 23, 2024 3:31:35 AM UTC
  • Commit: 4ee484519db441b6152dbac978fc3bfb9fbfd0a3
  • Branch: refs/heads/main

Release Note Category

  • Feature changes/additions
  • Bug fixes
  • Internal Infrastructure Improvements

Release Note Description

Update the OctoKit.NET library from 0.49.0 to 10.0.0

…d 20240222.1

CoreHealthMonitor , Microsoft.AspNetCore.ApiPagination , Microsoft.AspNetCore.ApiVersioning , Microsoft.AspNetCore.ApiVersioning.Analyzers , Microsoft.AspNetCore.ApiVersioning.Swashbuckle , Microsoft.DncEng.CommandLineLib , Microsoft.DncEng.Configuration.Extensions , Microsoft.DotNet.Authentication.Algorithms , Microsoft.DotNet.GitHub.Authentication , Microsoft.DotNet.Internal.DependencyInjection , Microsoft.DotNet.Internal.DependencyInjection.Testing , Microsoft.DotNet.Internal.Health , Microsoft.DotNet.Internal.Logging , Microsoft.DotNet.Internal.Testing.DependencyInjection.Abstractions , Microsoft.DotNet.Internal.Testing.DependencyInjectionCodeGen , Microsoft.DotNet.Internal.Testing.Utility , Microsoft.DotNet.Kusto , Microsoft.DotNet.Metrics , Microsoft.DotNet.ServiceFabric.ServiceHost , Microsoft.DotNet.Services.Utility , Microsoft.DotNet.Web.Authentication , ServiceFabricMocks
 From Version 1.1.0-beta.24115.1 -> To Version 1.1.0-beta.24122.1
…d 20240222.2

CoreHealthMonitor , Microsoft.AspNetCore.ApiPagination , Microsoft.AspNetCore.ApiVersioning , Microsoft.AspNetCore.ApiVersioning.Analyzers , Microsoft.AspNetCore.ApiVersioning.Swashbuckle , Microsoft.DncEng.CommandLineLib , Microsoft.DncEng.Configuration.Extensions , Microsoft.DotNet.Authentication.Algorithms , Microsoft.DotNet.GitHub.Authentication , Microsoft.DotNet.Internal.DependencyInjection , Microsoft.DotNet.Internal.DependencyInjection.Testing , Microsoft.DotNet.Internal.Health , Microsoft.DotNet.Internal.Logging , Microsoft.DotNet.Internal.Testing.DependencyInjection.Abstractions , Microsoft.DotNet.Internal.Testing.DependencyInjectionCodeGen , Microsoft.DotNet.Internal.Testing.Utility , Microsoft.DotNet.Kusto , Microsoft.DotNet.Metrics , Microsoft.DotNet.ServiceFabric.ServiceHost , Microsoft.DotNet.Services.Utility , Microsoft.DotNet.Web.Authentication , ServiceFabricMocks
 From Version 1.1.0-beta.24115.1 -> To Version 1.1.0-beta.24122.2
- see dotnet/dnceng#2065
- comment about Octokit.dll not being signed is no longer true (since v4.0.0)
  - still need to avoid signing Microsoft.DotNet.Maestro.Tasks.dll for other reasons
@dougbu dougbu force-pushed the darc-main-34ae2df5-12fa-4470-88f5-cfe19fa63458 branch from 11d9563 to f3ee316 Compare February 23, 2024 06:13
@dougbu dougbu enabled auto-merge (squash) February 23, 2024 06:15
@dougbu
Copy link
Member

dougbu commented Feb 23, 2024

doubt the bot can merge this due to my added commit. please approve 😁

…d 20240222.2

CoreHealthMonitor , Microsoft.AspNetCore.ApiPagination , Microsoft.AspNetCore.ApiVersioning , Microsoft.AspNetCore.ApiVersioning.Analyzers , Microsoft.AspNetCore.ApiVersioning.Swashbuckle , Microsoft.DncEng.CommandLineLib , Microsoft.DncEng.Configuration.Extensions , Microsoft.DotNet.Authentication.Algorithms , Microsoft.DotNet.GitHub.Authentication , Microsoft.DotNet.Internal.DependencyInjection , Microsoft.DotNet.Internal.DependencyInjection.Testing , Microsoft.DotNet.Internal.Health , Microsoft.DotNet.Internal.Logging , Microsoft.DotNet.Internal.Testing.DependencyInjection.Abstractions , Microsoft.DotNet.Internal.Testing.DependencyInjectionCodeGen , Microsoft.DotNet.Internal.Testing.Utility , Microsoft.DotNet.Kusto , Microsoft.DotNet.Metrics , Microsoft.DotNet.ServiceFabric.ServiceHost , Microsoft.DotNet.Services.Utility , Microsoft.DotNet.Web.Authentication , ServiceFabricMocks
 From Version 1.1.0-beta.24115.1 -> To Version 1.1.0-beta.24122.2
AlitzelMendez
AlitzelMendez previously approved these changes Feb 23, 2024
@premun premun changed the title [main] Update dependencies from dotnet/dnceng-shared Bump OctoKit to 10.0.0 Feb 23, 2024
@premun
Copy link
Member

premun commented Feb 23, 2024

Based on https://github.blog/changelog/2021-10-14-rest-api-preview-promotions/ I removed the non-existent header

premun
premun previously approved these changes Feb 23, 2024
dkurepa
dkurepa previously approved these changes Feb 23, 2024
@premun
Copy link
Member

premun commented Feb 23, 2024

The signing triggers a cascade of problems down the line. I will deal with that separately, let's just bump now and get it in

@premun premun requested a review from dkurepa February 23, 2024 11:14
@dougbu dougbu merged commit ad1622f into main Feb 23, 2024
9 checks passed
@dotnet-maestro dotnet-maestro bot deleted the darc-main-34ae2df5-12fa-4470-88f5-cfe19fa63458 branch February 23, 2024 11:36
@dougbu
Copy link
Member

dougbu commented Feb 23, 2024

The signing triggers a cascade of problems down the line. I will deal with that separately, let's just bump now and get it in

right, I had tried that before and restored the opt-out as a result. removed the comment b/c it wasn't correct and "to be consistent with every other non-test project" seemed useless. might have filed a follow-up after testing locally to see if everything worked without any opt-outs but it wasn't an immediate priority

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.

4 participants