Skip to content

Update 'px' to 'PX'#224

Merged
andy1li merged 2 commits intomainfrom
andy/fix-3
Oct 3, 2025
Merged

Update 'px' to 'PX'#224
andy1li merged 2 commits intomainfrom
andy/fix-3

Conversation

@andy1li
Copy link
Member

@andy1li andy1li commented Oct 2, 2025

Note

Switches SET expiration flag to PX in fixtures and refreshes golden outputs (paths, ports, IDs, hexdumps, ordering).

  • Expiry/commands:
    • Use uppercase PX for SET expiration in fixtures (was px).
  • Fixtures/logs:
    • Update paths from /tmp/... to /private/tmp/... in RDB-related fixtures.
    • Refresh recorded outputs: connection ports, replication IDs, RDB hexdumps, stream IDs/timestamps, and some key ordering.
    • No functional code changes; only test fixture/golden output updates to match current behavior.

Written by Cursor Bugbot for commit b37ce70. This will update automatically on new commits. Configure here.

… adjust timestamps in test outputs for consistency.
@andy1li andy1li self-assigned this Oct 2, 2025
@andy1li andy1li added the regenerate-fixtures Trigger a CI job to regenerate fixtures label Oct 2, 2025
@github-actions
Copy link

github-actions bot commented Oct 2, 2025

Triggered a Github Actions job to update fixtures.

@github-actions github-actions bot removed the regenerate-fixtures Trigger a CI job to regenerate fixtures label Oct 2, 2025
@andy1li andy1li requested a review from rohitpaulk October 2, 2025 12:58
@andy1li andy1li merged commit 10d8734 into main Oct 3, 2025
3 checks passed
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.

3 participants