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 github.com/microsoftgraph/msgraph-sdk-go to v1.46.0 #44205

Merged
merged 2 commits into from
Jul 15, 2024

Conversation

tigrato
Copy link
Contributor

@tigrato tigrato commented Jul 15, 2024

Versions v1.44.0 to v1.45.0 were recalled due to breaking changes in constructor names, as detailed in this pull request. These changes caused build failures for Teleport when using go.work.

This pull request updates the version in all affected go.mod files.

Version `v1.44.0`-`v1.45.0` were recall because they included breaking changes into constructures names microsoftgraph/msgraph-sdk-go#738 that cause teleport to fail builds when using `go.work`.

This PR bumps the version to all affected go.mod files.

Signed-off-by: Tiago Silva <[email protected]>
Copy link

The PR changelog entry failed validation: Changelog entry not found in the PR body. Please add a "no-changelog" label to the PR, or changelog lines starting with changelog: followed by the changelog entries for the PR.

@tigrato tigrato added the no-changelog Indicates that a PR does not require a changelog entry label Jul 15, 2024
@tigrato
Copy link
Contributor Author

tigrato commented Jul 15, 2024

We need to ensure we never update /integrations/*/go.mod when we do not update /go.mod as this causes annoying problems.

@@ -146,7 +146,7 @@ require (
github.com/mdlayher/netlink v1.7.2
github.com/microsoft/go-mssqldb v1.7.2 // replaced
github.com/microsoft/kiota-authentication-azure-go v1.0.2
github.com/microsoftgraph/msgraph-sdk-go v1.43.0
github.com/microsoftgraph/msgraph-sdk-go v1.46.0
Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks. I noticed msgraph-sdk-go was weird in our last update batch and kept it at v1.43.0 (#43828 (comment)), but apparently some of the indirects in other modules skipped. I figured a fix would land at some point.

@codingllama
Copy link
Contributor

It looks like you need to tidy the root module.

@tigrato tigrato enabled auto-merge July 15, 2024 14:15
@tigrato tigrato added this pull request to the merge queue Jul 15, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 15, 2024
@tigrato tigrato added this pull request to the merge queue Jul 15, 2024
Merged via the queue into master with commit 16fedf3 Jul 15, 2024
40 checks passed
@tigrato tigrato deleted the tigrato/ms-graph-recall branch July 15, 2024 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-changelog Indicates that a PR does not require a changelog entry size/sm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants