enhance: add latest revision to write options #14

Merged

thedadams merged 1 commit from latest-revision-conflict into main

2025-02-11 15:19:22 +00:00

thedadams commented

2025-02-11 15:18:08 +00:00

(Migrated from github.com)

If the latest revision write option is passed and this is not the latest revision, then writing the file will fail with a conflict error. If the latest revision write option is not passed, then no check is done and writing will not fail with a conflict.

copilot-pull-request-reviewer[bot] (Migrated from github.com) reviewed

2025-02-11 15:20:17 +00:00

copilot-pull-request-reviewer[bot] (Migrated from github.com) left a comment

Copilot reviewed 5 out of 11 changed files in this pull request and generated no comments.

Files not reviewed (6)

tool.gpt: Language not supported
pkg/client/fileinfo.go: Evaluated as low risk
pkg/server/writefile.go: Evaluated as low risk
pkg/client/client.go: Evaluated as low risk
pkg/client/revisions.go: Evaluated as low risk
pkg/client/directory.go: Evaluated as low risk

Comments suppressed due to low confidence (1)

pkg/client/s3.go:188

Ensure that the new behavior introduced by the LatestRevision option is covered by tests.

if opt.LatestRevision != "" {

Copilot reviewed 5 out of 11 changed files in this pull request and generated no comments. <details> <summary>Files not reviewed (6)</summary> * **tool.gpt**: Language not supported * **pkg/client/fileinfo.go**: Evaluated as low risk * **pkg/server/writefile.go**: Evaluated as low risk * **pkg/client/client.go**: Evaluated as low risk * **pkg/client/revisions.go**: Evaluated as low risk * **pkg/client/directory.go**: Evaluated as low risk </details> <details> <summary>Comments suppressed due to low confidence (1)</summary> **pkg/client/s3.go:188** * Ensure that the new behavior introduced by the `LatestRevision` option is covered by tests. ``` if opt.LatestRevision != "" { ``` </details>