Skip to content

Add GitHub Actions Build#295

Merged
JoniVR merged 23 commits intoMonitorControl:masterfrom
stephengroat:github-actions
Aug 4, 2021
Merged

Add GitHub Actions Build#295
JoniVR merged 23 commits intoMonitorControl:masterfrom
stephengroat:github-actions

Conversation

@stephengroat
Copy link
Copy Markdown
Contributor

Note: This does change the Debug scheme to use locally signed certs instead of the team cert. This shouldn't affect release builds

Comment thread .github/workflows/main.yml Outdated
push:
branches: [ master ]
pull_request:
branches: [ master ]
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think this only runs when the pull request head branch is master, which is why it isn't working for this pull request.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

works on my fork pr https://github.com/stephengroat/MonitorControl/pull/1, maybe it has to be merged in?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yes I think that this should work, CI should run when changes are merged into master and for each PR. It's probably indeed due to needing to be merged in first.

Comment thread .github/workflows/main.yml Outdated
@JoniVR JoniVR self-requested a review June 15, 2021 20:06
Copy link
Copy Markdown
Member

@JoniVR JoniVR left a comment

Choose a reason for hiding this comment

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

I opened up a test PR on your branch (https://github.com/stephengroat/MonitorControl/pull/2), seems like it fails the build now throwing a linking error for Swiftlint.

My best guess is it's probably the brew link step that can go since SwiftLint seems to be included by default on macOS for Github Actions? Not sure.

@stephengroat
Copy link
Copy Markdown
Contributor Author

@JoniVR unfortunately, Preferences now requires swift-tools-version 5.4 which is xcode 12.5, only available on macos-11.

I've asked for access to the private preview, I'll have to see what happens

@JoniVR
Copy link
Copy Markdown
Member

JoniVR commented Jun 15, 2021

Hi, thanks for your effort, it's much appreciated. I did some reading and found the following:

They state it might take 2-3 days to propagate so that would hopefully solve it then.

@AlenaSviridenko
Copy link
Copy Markdown

Hi @JoniVR,
VE maintainer here, sorry for pinging you here, but I saw several requests for this org to join macOS 11 pool, and we definitely can do that, but it requires additional action from your side.
I have sent a couple of clarifying emails back in June and now, but they might have land to spam folder. Please, check them and after that I will be able to provide access to the macOS 11 pool.
Thanks.

@JoniVR
Copy link
Copy Markdown
Member

JoniVR commented Aug 4, 2021

Hi, thanks for pinging me here. So our best bet seems to be just creating a workflow (any workflow) that triggers Github Actions, after that it should work if I read everything correctly?

JoniVR pushed a commit that referenced this pull request Aug 4, 2021
@AlenaSviridenko
Copy link
Copy Markdown

Correct, you can try it with macos-latest, even if it fails, it will still trigger the process

@JoniVR
Copy link
Copy Markdown
Member

JoniVR commented Aug 4, 2021

I ran a CI action from a separate branch, is that sufficient? The actions tab seems to indicate that it ran.

@AlenaSviridenko
Copy link
Copy Markdown

Yes, confirming that I can find the required key now, will provide access today. Thanks!

@JoniVR
Copy link
Copy Markdown
Member

JoniVR commented Aug 4, 2021

Thanks for the help, much appreciated!

@AlenaSviridenko
Copy link
Copy Markdown

macOS 11 should be accessible for you now, please check, and let me know if it doesn't work 🙂

@JoniVR JoniVR merged commit 40d557b into MonitorControl:master Aug 4, 2021
@JoniVR
Copy link
Copy Markdown
Member

JoniVR commented Aug 4, 2021

Thanks for the PR @stephengroat!

@stephengroat stephengroat deleted the github-actions branch August 11, 2021 19:23
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.

4 participants