-
Notifications
You must be signed in to change notification settings - Fork 6
Migrate to Rest API v0.7 #25
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
Changes from all commits
aad6221
3be9333
9c39e7a
526ceaf
82f79a1
9e5e84b
eb5d0ae
21b8f7d
2912dc2
72e3089
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -1,3 +1,29 @@ | ||||||
| ## 2.0.0 | ||||||
|
|
||||||
| BREAKING CHANGES: | ||||||
|
|
||||||
| * Target REST API v0.7 (previously v0.5) | ||||||
| * Remove `ImageInfo` and `VideoMeta` fields from `file.BasicFileInfo` — use `ContentInfo.Image` and `ContentInfo.Video` | ||||||
| * Remove `RecognitionInfo` field from `file.Info` — use `AppData` | ||||||
| * Add `Metadata` and `AppData` fields to `file.Info` | ||||||
| * Remove `group.Store()` method (endpoint removed in v0.7) | ||||||
| * Remove `file.Copy()` method and `file.CopyParams` type — use `LocalCopy()` and `RemoteCopy()` | ||||||
| * Remove `file.OrderBySizeAsc` and `file.OrderBySizeDesc` constants (not supported in v0.7) | ||||||
| * Remove `APIv05` and `APIv06` constants | ||||||
| * Minimum Go version is now 1.25 | ||||||
|
Comment on lines
+5
to
+13
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Good practice to group your actions.
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Good point, will organize it before 2.0.0 release
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
|
|
||||||
| IMPROVEMENTS: | ||||||
|
|
||||||
| * Add `UserAgent` field to `ucare.Config` for custom agent identification | ||||||
| * Replace `http.NewRequest` + `WithContext` with `http.NewRequestWithContext` | ||||||
| * Throttle retry loops now respect context cancellation | ||||||
| * Replace `ioutil` usage with `io` equivalents | ||||||
| * Replace `go-env` dependency with `os.Getenv` | ||||||
| * Update `stretchr/testify` to v1.10.0 | ||||||
| * Update CI: Go 1.25, modern GitHub Actions versions, remove deprecated golint | ||||||
| * Integration tests skip gracefully when credentials are not set | ||||||
| * Fix errors in package documentation examples | ||||||
|
|
||||||
| ## 1.2.1 (September 1, 2020) | ||||||
|
|
||||||
| IMPROVEMENTS: | ||||||
|
|
||||||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,18 +1,13 @@ | ||
| GO ?= go | ||
|
|
||
| vet: | ||
| @$(GO) vet ./... | ||
| .PHONY: vet | ||
|
|
||
| lint: vet | ||
| @golint -set_exit_status ./... | ||
| .PHONY: lint | ||
|
|
||
| test: lint | ||
| test: vet | ||
| @$(GO) test -v -race -short ./... | ||
| .PHONY: test | ||
|
|
||
| test-full: lint | ||
| test-full: vet | ||
| @$(GO) test -v -race ./... | ||
| .PHONY: test-full | ||
|
|
||
| vet: | ||
| @$(GO) vet ./... | ||
| .PHONY: vet |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.