Skip to content

Remove sync change requirement#204

Merged
connorgallopo merged 11 commits into
mainfrom
feature/sync-change
May 20, 2026
Merged

Remove sync change requirement#204
connorgallopo merged 11 commits into
mainfrom
feature/sync-change

Conversation

@connorgallopo
Copy link
Copy Markdown
Collaborator

Description

What does this PR do?

Why is this change needed?

Related Issue(s)

Closes #

Changes Made

Testing Performed

Test Environment

  • Pi Model:
  • Camera Type:
  • OS Version:
  • Installation Method:

Test Results

  • pitrac test hardware passes
  • pitrac test camera passes (if camera-related)
  • pitrac test pulse passes (if strobe-related)
  • Simulator integration tested with:
  • Performance metrics:

Test Commands Run

# Paste the actual commands and output

Performance Impact

  • Before:
  • After:
  • Impact:

Breaking Changes

  • This PR includes breaking changes
    • What breaks:
    • Migration steps:

Dependencies

  • No new dependencies
  • [ ] New dependencies added:

  • [ ] Updated dependencies:

Hardware Compatibility

  • Tested on Pi 5
  • Tested on Pi 4
  • Tested with single Pi setup
  • Tested with dual Pi setup

Documentation

  • No documentation needed
  • Documentation updated in this PR
  • Documentation PR to follow
  • Updated relevant sections:
    • README
    • Hardware guide
    • Software guide
    • Troubleshooting guide

Screenshots/Videos

AI (Vibe-Coded) Content Description

Checklist

Code Quality

  • Code follows existing patterns and conventions
  • No unnecessary comments added
  • Error handling implemented appropriately

Build & Test

  • Successfully builds with ./packaging/build.sh build
  • All existing tests pass
  • New tests added for new functionality
  • Tested on actual Raspberry Pi hardware (not just CI)

Submission Requirements

  • Commits squashed if needed (git rebase -i HEAD~n)
  • CLA signed
  • PR title follows format: [PR TYPE] Brief description
  • Branch is up-to-date with main

Additional Context


@connorgallopo connorgallopo force-pushed the feature/sync-change branch from 87649e2 to 3122d54 Compare May 14, 2026 15:34
@connorgallopo connorgallopo force-pushed the feature/sync-change branch from 3122d54 to 9ee6498 Compare May 18, 2026 16:32
@connorgallopo connorgallopo changed the title Ran eslint --fix across JS files Remove sync change requirement May 20, 2026
@connorgallopo connorgallopo force-pushed the feature/sync-change branch from c59ba39 to c153c1b Compare May 20, 2026 17:16
@connorgallopo connorgallopo marked this pull request as ready for review May 20, 2026 17:17
@connorgallopo connorgallopo merged commit 970796c into main May 20, 2026
8 checks passed
@connorgallopo connorgallopo deleted the feature/sync-change branch May 20, 2026 17:23
@github-actions github-actions Bot locked and limited conversation to collaborators May 20, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant