Skip to content

Use Rock remote cache status - #26

Draft
thymikee wants to merge 1 commit into
mainfrom
agent/remote-cache-status
Draft

Use Rock remote cache status#26
thymikee wants to merge 1 commit into
mainfrom
agent/remote-cache-status

Conversation

@thymikee

Copy link
Copy Markdown
Member

Summary

Use Rock 0.15.1's remote-cache status command for cache lookup.

This removes the action's separate fingerprint lookup and manual artifact-name construction, while preserving PR-specific lookup before the regular cache lookup. Rock now provides the canonical fingerprint, artifact name, and hit result. The documented minimum Rock version is now 0.15.1.

Validation

  • Parsed action.yml as YAML
  • Verified the status JSON parsing for S3 hit and miss responses, including artifacts without an id
  • End-to-end validation will use the PR commit in Expensify's Android workflow

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.

1 participant