feat: Rewrite in Go plus full api coverage and OAuth support#281
feat: Rewrite in Go plus full api coverage and OAuth support#281drstrangelooker wants to merge 36 commits into
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request implements a complete Go-based rewrite of the Looker Content Utility (gzr), replacing the original Ruby codebase. It introduces a robust CLI using Cobra, supporting Looker API 4.0 resource management and OAuth PKCE authentication. The review feedback highlights several critical issues, including compilation errors due to missing pointer dereferences and type mismatches in alert.go, attribute.go, dashboard.go, and space.go. Additionally, manual HTTP requests in project_files.go bypass authentication headers, the sudo logic is bypassed for direct tokens in client.go, and token_file.go requires explicit permission enforcement to prevent insecure file permissions.
CI Tests 4 files 24 suites 0s ⏱️ Results for commit 31c330d. ♻️ This comment has been updated with latest results. |
drstrangelooker
force-pushed
the
go
branch
5 times, most recently
from
5502e43 to
b125580
Compare
|
@joshtemple I addressed everything Gemini came up with. You can pull this branch and build with |
1 participant
Release-As: 0.4.0