Skip to content

fix: explicitly checkout flutter-client-sdk repo in update-ios-sdk-version workflow#245

Open
jonathannorris wants to merge 3 commits into
mainfrom
fix/explicitly-checkout-flutter-client-sdk-in-ios-update-workflow
Open

fix: explicitly checkout flutter-client-sdk repo in update-ios-sdk-version workflow#245
jonathannorris wants to merge 3 commits into
mainfrom
fix/explicitly-checkout-flutter-client-sdk-in-ios-update-workflow

Conversation

@jonathannorris
Copy link
Copy Markdown
Member

Summary

When called as a reusable workflow from ios-client-sdk, actions/checkout defaults to checking out the calling repo. The ios/devcycle_flutter_client_sdk.podspec file doesn't exist there, causing the sed update step to fail with exit code 1. This has been happening on every scheduled Monday run and every iOS release trigger since the workflow was introduced.

Copilot AI review requested due to automatic review settings May 15, 2026 21:01
@jonathannorris jonathannorris requested a review from a team as a code owner May 15, 2026 21:01
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes the iOS SDK version update reusable workflow so it checks out the Flutter client SDK repository rather than the calling repository when invoked cross-repo.

Changes:

  • Adds an explicit repository: DevCycleHQ/flutter-client-sdk setting to the iOS workflow checkout step.
  • Aligns the iOS workflow with the previously fixed Android SDK update workflow behavior.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI review requested due to automatic review settings May 15, 2026 21:10
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

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.

2 participants