Skip to content

Conversation

@ssestak
Copy link
Contributor

@ssestak ssestak commented Nov 13, 2025

Summary

This PR adds Git LFS support to the universal-detect-changes-and-generate-changelog action, enabling workflows to download LFS files during the initial checkout phase.

Key Changes

  • Action Enhancement: Added use_git_lfs input parameter to the changelog generation action
  • Workflow Optimization: Removed redundant checkout steps from iOS nightly workflow
  • Condition Simplification: Simplified checkout logic in on-demand workflow

Benefits

  • ✅ LFS files available during entire workflow execution
  • ✅ Eliminates duplicate checkout operations
  • ✅ Cleaner workflow logic
  • ✅ Backward compatible (default: false)

Implementation Details

universal-detect-changes-and-generate-changelog action:

  • New use_git_lfs boolean input (default: false)
  • LFS parameter passed to actions/checkout@v4

iOS Nightly Build:

  • Removed conditional checkout step (now handled by action)
  • LFS parameter propagated to action

iOS On-Demand Build:

  • Simplified checkout condition to if: inputs.changelog != ''
  • LFS parameter propagated to action

Test Coverage

All scenarios verified:

  • ✅ LFS enabled + generated changelog
  • ✅ LFS enabled + custom changelog
  • ✅ LFS disabled + generated changelog
  • ✅ LFS disabled + custom changelog

🤖 Generated with Claude Code

Propagates the use_git_lfs input parameter from iOS workflows to the
universal-detect-changes-and-generate-changelog action, enabling LFS
files to be downloaded during the initial checkout when needed.

This eliminates the need for duplicate checkout steps and ensures LFS
files are available throughout the entire workflow execution.

Changes:
- Add use_git_lfs input parameter to changelog action
- Pass LFS parameter to checkout in the action
- Propagate LFS input from iOS workflows to the action
- Remove redundant checkout step from nightly workflow
- Simplify on-demand workflow checkout condition

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

Co-Authored-By: Claude <[email protected]>
@jmarek41 jmarek41 merged commit b498217 into main Nov 13, 2025
2 checks passed
@jmarek41 jmarek41 deleted the fix/iOS-LFS branch November 13, 2025 14:38
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