go-gitea/lgtm
1
0
Fork 0
mirror of https://github.com/go-gitea/lgtm.git synced 2026-07-01 21:34:01 -04:00
Code Issues 6 Packages Projects Releases 1 Wiki Activity

[PR #36] [MERGED] Upgrade GitHub API to V3 #37

New Issue
Closed
opened 2026-02-15 16:29:15 -05:00 by yindo · 0 comments
yindo commented 2026-02-15 16:29:15 -05:00
Owner
Copy Link
Copy Source

📋 Pull Request Information

Original PR: https://github.com/go-gitea/lgtm/pull/36
Author: @appleboy
Created: 12/27/2017
Status: ✅ Merged
Merged: 12/28/2017
Merged by: @lunny

Base: master ← Head: upgrade

📝 Commits (9)

📊 Changes

100 files changed (+19999 additions, -1842 deletions)

View changed files

📝 .drone.yml (+33 -27)
➖ .drone.yml.sig (+0 -1)
📝 .gitignore (+2 -0)
📝 Dockerfile (+7 -9)
📝 Dockerfile.rpi (+1 -1)
📝 Makefile (+2 -2)
📝 api/teams_test.go (+0 -25)
📝 cache/helper_test.go (+12 -12)
📝 main.go (+3 -4)
📝 model/hook.go (+1 -0)
➕ model/review.go (+17 -0)
📝 remote/github/client.go (+16 -7)
📝 remote/github/github.go (+118 -60)
📝 remote/github/types.go (+48 -7)
📝 remote/github/utils.go (+22 -15)
📝 remote/mock/remote.go (+118 -90)
📝 remote/remote.go (+40 -32)
➕ vendor/github.com/google/go-github/CONTRIBUTING.md (+114 -0)
➕ vendor/github.com/google/go-github/LICENSE (+341 -0)
➕ vendor/github.com/google/go-github/README.md (+244 -0)

...and 80 more files

📄 Description

  1. Upgrade GitHub API to V3
  2. Support LGTM from review approve

See the PR example

@go-gitea/maintainers you can try it on https://lgtm.wu-boy.com/

The old LGTM website can't enable any repository now. https://lgtm.gitea.io/

screen shot 2017-12-26 at 10 48 40 pm

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/go-gitea/lgtm/pull/36 **Author:** [@appleboy](https://github.com/appleboy) **Created:** 12/27/2017 **Status:** ✅ Merged **Merged:** 12/28/2017 **Merged by:** [@lunny](https://github.com/lunny) **Base:** `master` ← **Head:** `upgrade` --- ### 📝 Commits (9) - [`e96596f`](https://github.com/go-gitea/lgtm/commit/e96596f8bffcbbeeedf491c98c8ffa0bde8980f4) feat: upgrade to github API v3. - [`ccd12dc`](https://github.com/go-gitea/lgtm/commit/ccd12dc359040ec6e53ea98aa673429c70800b22) update drone - [`ef2b358`](https://github.com/go-gitea/lgtm/commit/ef2b3588fdda82440820d984ebf250d8ce13ecbd) fix - [`4f5e50c`](https://github.com/go-gitea/lgtm/commit/4f5e50c7a0a9a07f557909aa0d1bb420133eba0c) fix: testing - [`77f0c0e`](https://github.com/go-gitea/lgtm/commit/77f0c0e382de8ad462eaa23a5d2373134934e7ca) fix: remove discord token - [`313b0c5`](https://github.com/go-gitea/lgtm/commit/313b0c5e1d0472cab18e352214542f06f2fad981) fix: secrets. - [`46db21c`](https://github.com/go-gitea/lgtm/commit/46db21c135e282b3a67d01918592322b12d45641) fix: drone secrets - [`fe5f415`](https://github.com/go-gitea/lgtm/commit/fe5f415be1b73d6af4248433bf0dfd790bff90d4) update codecov condition. - [`b13eac2`](https://github.com/go-gitea/lgtm/commit/b13eac2693e3b14fa744f1a3dc4e93f24dc9a2d6) update dockerfile ### 📊 Changes **100 files changed** (+19999 additions, -1842 deletions) <details> <summary>View changed files</summary> 📝 `.drone.yml` (+33 -27) ➖ `.drone.yml.sig` (+0 -1) 📝 `.gitignore` (+2 -0) 📝 `Dockerfile` (+7 -9) 📝 `Dockerfile.rpi` (+1 -1) 📝 `Makefile` (+2 -2) 📝 `api/teams_test.go` (+0 -25) 📝 `cache/helper_test.go` (+12 -12) 📝 `main.go` (+3 -4) 📝 `model/hook.go` (+1 -0) ➕ `model/review.go` (+17 -0) 📝 `remote/github/client.go` (+16 -7) 📝 `remote/github/github.go` (+118 -60) 📝 `remote/github/types.go` (+48 -7) 📝 `remote/github/utils.go` (+22 -15) 📝 `remote/mock/remote.go` (+118 -90) 📝 `remote/remote.go` (+40 -32) ➕ `vendor/github.com/google/go-github/CONTRIBUTING.md` (+114 -0) ➕ `vendor/github.com/google/go-github/LICENSE` (+341 -0) ➕ `vendor/github.com/google/go-github/README.md` (+244 -0) _...and 80 more files_ </details> ### 📄 Description 1. Upgrade GitHub API to V3 2. Support `LGTM` from [review approve](https://developer.github.com/v3/pulls/reviews/) See the [PR example](https://github.com/go-training/drone-golang-example/pull/4) @go-gitea/maintainers you can try it on https://lgtm.wu-boy.com/ The old `LGTM` website can't enable any repository now. https://lgtm.gitea.io/ <img width="870" alt="screen shot 2017-12-26 at 10 48 40 pm" src="https://user-images.githubusercontent.com/21979/34358947-11918236-ea8f-11e7-8b19-58c304a77cf7.png"> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-15 16:29:15 -05:00
yindo closed this issue 2026-02-15 16:29:15 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
hacktoberfest
kind/bug
kind/docs
kind/enhancement
pull-request
Mirrored from GitHub Pull Request
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: go-gitea/lgtm#37
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?