Skip to content

Conversation

rasendubi
Copy link
Collaborator

@rasendubi rasendubi commented Nov 28, 2024

@typotter I would look for an approach like this for #73

Basically:

  • override submodule checkout in ci workflow
  • don't touch sdk-test-data in the rest of workflows

Comment on lines 31 to +43
steps:
- uses: actions/checkout@v4
with:
repository: Eppo-exp/eppo-multiplatform
ref: ${{ inputs.sdk_branch || github.ref }}
submodules: true

- uses: actions/checkout@v4
if: ${{ inputs.test_data_branch }}
with:
repository: Eppo-exp/sdk-test-data
ref: ${{ inputs.test_data_branch }}
path: sdk-test-data
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I hope that this should be enough but maybe we need to override submodules to false if test_data_branch is supplied

@rasendubi rasendubi closed this Dec 3, 2024
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.

1 participant