Skip to content

v0.2

  • v0.2
  • 84e447c

  • Verified

    This commit was created on GitHub.com and signed with GitHub’s verified signature.
    GPG key ID: B5690EEEBB952194
    Verified
    Learn about vigilant mode

  • Choose a tag to compare

    View all tags
v0.2: Fix BDD tests for Go implementation (#13)
  • v0.2
  • 84e447c

  • Choose a tag to compare

    View all tags

  • Verified

    This commit was created on GitHub.com and signed with GitHub’s verified signature.
    GPG key ID: B5690EEEBB952194
    Verified
    Learn about vigilant mode
@DamianReeves DamianReeves tagged this 29 Aug 19:50 
* Fix CI build by excluding broken BDD tests from default test target

- Update Makefile test target to exclude test/bdd packages from default runs
- BDD tests still available via 'make test-bdd' target for manual testing
- Update check target to exclude BDD tests and prevent CI failures
- This allows CI to pass while BDD tests are being fixed/updated for Go implementation

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

* Complete BDD test fixes for Go implementation

- Fix exit code handling in BDD tests using proper exec.ExitError checking
- Add helper runCommand method to consistently handle command execution and output
- Update deprecated godog Before/After scenario hooks to current API
- Fix binary path detection for both test/bdd and project root execution contexts
- Implement missing step definitions for all undefined scenarios
- Update output validation to match Go implementation's log format
- Re-enable BDD tests in default test target and check target

All 13 scenarios now pass with 80 steps, providing comprehensive test coverage for:
- Basic sync operations (one-way, two-way, dry-run)
- Git patch generation and application workflows
- Ignore pattern handling (.syncignore, .gitignore)
- Interactive command validation

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

* Fix deprecated GitHub Actions in CI workflow

- Update actions/upload-artifact from v3 to v4 to resolve deprecation warning
- Update actions/cache from v3 to v4 for consistency with latest versions
- Resolves CI failure: "This request has been automatically failed because it uses a deprecated version of actions/upload-artifact: v3"

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

* fix: correct release drafter config file path

Fix release drafter workflow by removing .github/ prefix from config-name.
The release-drafter action already looks in the .github/ directory by default.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

* remove: ubuntu-20.04 from CI matrix

Remove ubuntu-20.04 from the rsync matrix workflow to focus testing
on more recent Ubuntu versions (22.04 and latest).

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

* remove: obsolete rsync-matrix workflow

Remove rsync-matrix.yml workflow as it was designed for Python implementation
and is no longer needed. The main ci.yml workflow properly tests the Go version.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

---------

Co-authored-by: Claude <noreply@anthropic.com>
Assets 2
Loading