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

Updates go-github to v54 for projects v2 support #771

Merged
merged 2 commits into from
Apr 29, 2024
Merged

Updates go-github to v54 for projects v2 support #771

merged 2 commits into from
Apr 29, 2024

Conversation

dsspence
Copy link
Contributor

@dsspence dsspence commented Apr 29, 2024

Summary

This PR bumps up the major version of the google/go-github dependency from v41 to v54.

While this isn't the latest version, it is the first version that introduces support for the Projects v2 API that could be used in creating subscriptions #409.

The only breaking change I could identify was the use of a new wrapper datatype Timestamp, where previously time.Time was being used.

I don't have my E2E tests setup locally yet but can review the CI run when a maintainer allows it.

Ticket Link

@mattermost-build
Copy link
Contributor

Hello @dsspence,

Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

Copy link
Contributor

@hanzei hanzei left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution @dsspence !

go.mod Outdated Show resolved Hide resolved
go.sum Outdated Show resolved Hide resolved
@dsspence dsspence requested a review from hanzei April 29, 2024 13:44
Copy link
Contributor

@hanzei hanzei left a comment

Choose a reason for hiding this comment

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

LGTM

@hanzei hanzei added the 2: Dev Review Requires review by a core committer label Apr 29, 2024
@mickmister mickmister added 4: Reviews Complete All reviewers have approved the pull request and removed 2: Dev Review Requires review by a core committer labels Apr 29, 2024
@mickmister mickmister merged commit dbedde7 into mattermost:master Apr 29, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4: Reviews Complete All reviewers have approved the pull request Contributor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants