Skip to content

Switch to GitHub Actions#794

Merged
djbe merged 8 commits intodevelopfrom
feature/github-actions
Apr 11, 2021
Merged

Switch to GitHub Actions#794
djbe merged 8 commits intodevelopfrom
feature/github-actions

Conversation

@djbe
Copy link
Copy Markdown
Member

@djbe djbe commented Oct 12, 2020

Mirror to SwiftGen/StencilSwiftKit#129, review that first.

@AliSoftware I think you're the only one with access to @SwiftGen-Eve, so you'll probably have to create a new Danger token.

@djbe djbe force-pushed the feature/github-actions branch 3 times, most recently from df5f8ba to f29e45a Compare October 15, 2020 11:18
@djbe djbe force-pushed the feature/github-actions branch 2 times, most recently from cbcec80 to fb23ea4 Compare October 26, 2020 00:25
@djbe djbe marked this pull request as ready for review October 26, 2020 00:27
@djbe
Copy link
Copy Markdown
Member Author

djbe commented Oct 26, 2020

Weirdly enough the SPM macOS step is failing here, even though it's working for StencilSwiftKit: https://github.com/SwiftGen/StencilSwiftKit/runs/1305878101?check_suite_focus=true

I can't really see a major difference between the 2, except the all_xcode_versions rakelib code.
In SSK:

xcodes = `mdfind "kMDItemCFBundleIdentifier = 'com.apple.dt.Xcode'"`.chomp.split("\n")`

In SwiftGen:

mdfind_xcodes, _, _ = Open3.capture3('mdfind', "kMDItemCFBundleIdentifier = 'com.apple.dt.Xcode'")
xcodes = mdfind_xcodes.chomp.split("\n")

Edit: weeeeird, seems the SSK version fixes it? 😕 @AliSoftware you made this change in 1a12415, any idea what went wrong there?

@djbe djbe requested a review from AliSoftware October 26, 2020 00:47
@SwiftGen-Eve
Copy link
Copy Markdown

SwiftGen-Eve commented Oct 26, 2020

Hey 👋 I'm Eve, the friendly bot watching over SwiftGen 🤖

Thanks a lot for your contribution!


Seems like everything is in order 👍 You did a good job here! 🤝

Generated by 🚫 Danger

@djbe
Copy link
Copy Markdown
Member Author

djbe commented Oct 26, 2020

Welcome back Eve 🎉

@djbe djbe force-pushed the feature/github-actions branch from 1218852 to 90e5236 Compare April 11, 2021 19:04
@djbe djbe force-pushed the feature/github-actions branch from 90e5236 to f3ca62e Compare April 11, 2021 19:16
@djbe djbe merged commit b014b64 into develop Apr 11, 2021
@djbe djbe deleted the feature/github-actions branch April 11, 2021 19:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants