Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move to Buildkite #139

Merged
merged 13 commits into from
Jul 6, 2022
Merged

Move to Buildkite #139

merged 13 commits into from
Jul 6, 2022

Conversation

jkmassel
Copy link
Collaborator

@jkmassel jkmassel commented Jun 29, 2022

Use Buildkite instead of CircleCI for PR tests.

This PR also adds automated releases when a tag is made.

@bjtitus
Copy link
Collaborator

bjtitus commented Jun 30, 2022

@jkmassel Do I need to do anything special to access the build results? I'm getting a 404 but I do see other build pipelines, including WordPress iOS, at https://buildkite.com/automattic.

Copy link
Contributor

@mokagio mokagio left a comment

Choose a reason for hiding this comment

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

GitHub already shows that all the CI checks were successful, but for reference here's a screenshot of the build

image

fastlane/report.xml

# CocoaPods
Example/Pods
Copy link
Contributor

@mokagio mokagio Jul 6, 2022

Choose a reason for hiding this comment

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

I can see the the Pods/ folder is ignored here. However, this folder must have been tracked before this line was added because it appears in the diff.

Untitled3

I'm going to take the liberty to remove it then merge this PR. I'm confident in doing this because, in case I misunderstood something and we actually want to check the Pods into the repo, we can always add a new local copy and push it to the remote.

This folder is meant to be ignored, see `.gitignore` but must have
snuck into the repo before that instruction was committed.
@mokagio
Copy link
Contributor

mokagio commented Jul 6, 2022

@bjtitus

Do I need to do anything special to access the build results? I'm getting a 404...

You are now part of the team that has access to this pipeline in Buildkite, and should be able to see it (e.g. here). Please let me know if that's not the case. Thank you.

@mokagio mokagio merged commit 78f8f86 into main Jul 6, 2022
@mokagio mokagio deleted the move/to-buildkite branch July 6, 2022 06:40
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.

3 participants