Skip to content

ci(bindings/swift): add CI for Swift binding#2492

Merged
Xuanwo merged 4 commits intoapache:mainfrom
unixzii:swift-ci
Jun 20, 2023
Merged

ci(bindings/swift): add CI for Swift binding#2492
Xuanwo merged 4 commits intoapache:mainfrom
unixzii:swift-ci

Conversation

@unixzii
Copy link
Copy Markdown
Contributor

@unixzii unixzii commented Jun 18, 2023

No description provided.

@github-actions github-actions Bot added the releases-note/ci The PR modifies CI-related content or has a title that begins with "ci" label Jun 18, 2023
@unixzii unixzii mentioned this pull request Jun 18, 2023
6 tasks
@Xuanwo
Copy link
Copy Markdown
Member

Xuanwo commented Jun 19, 2023

Hi, ASF projects has strict limitation on actions that we can use:

swift-actions/setup-swift@v1 is not allowed to be used in apache/incubator-opendal.

We will need to ask for the ASF Infra to enable this action, please wait for sometime.

@unixzii
Copy link
Copy Markdown
Contributor Author

unixzii commented Jun 19, 2023

Got it, thanks.

@Xuanwo
Copy link
Copy Markdown
Member

Xuanwo commented Jun 19, 2023

Jira ticket created: https://issues.apache.org/jira/browse/INFRA-24710

@Xuanwo
Copy link
Copy Markdown
Member

Xuanwo commented Jun 20, 2023

Hi, swift-actions/setup-swift has been enabled. Please trigger to CI to give it a test. Thanks!

@unixzii
Copy link
Copy Markdown
Contributor Author

unixzii commented Jun 20, 2023

Hi @Xuanwo, it seems the test results are promising. I think we can merge it now.

Copy link
Copy Markdown
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Thanks!

@Xuanwo Xuanwo merged commit 08bb33a into apache:main Jun 20, 2023
@PsiACE PsiACE mentioned this pull request Jun 27, 2023
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

releases-note/ci The PR modifies CI-related content or has a title that begins with "ci"

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants