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

Convert to modules, upgrade dependencies #186

Merged
merged 1 commit into from
May 12, 2020
Merged

Conversation

bluekeyes
Copy link
Member

Switch from dep to Go modules and upgrade palantir/go-githubapp,
palantir/go-baseapp, and google/go-github. The go-github upgrade
required inlining some functions for team operations that were removed
due to API changes on GitHub.com. GitHub still supports the old APIs and
the new APIs have not shipped in an enterprise release yet.

Switch from dep to Go modules and upgrade palantir/go-githubapp,
palantir/go-baseapp, and google/go-github. The go-github upgrade
required inlining some functions for team operations that were removed
due to API changes on GitHub.com. GitHub still supports the old APIs and
the new APIs have not shipped in an enterprise release yet.
// TODO(bkeyes): this is a temporary workarount to add functions for APIs that
// are still supported by GitHub but were removed by google/go-github in
// v29.0.3. The replacement methods have not shipped in a GitHub Enterprise
// release yet, so we can't start using them.
Copy link
Member

Choose a reason for hiding this comment

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

:/

@bluekeyes bluekeyes merged commit 67b611a into develop May 12, 2020
@bluekeyes bluekeyes deleted the bkeyes/go-modules branch May 12, 2020 18:12
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.

2 participants