Skip to content
This repository was archived by the owner on Feb 5, 2025. It is now read-only.

Conversation

@mokagio
Copy link
Contributor

@mokagio mokagio commented Aug 4, 2022

Sets up Test Analytics.

image

image

@mokagio mokagio requested a review from crazytonyli August 4, 2022 06:33
@mokagio mokagio marked this pull request as ready for review August 4, 2022 06:33
@mokagio mokagio enabled auto-merge August 4, 2022 06:33
buildlog_path: File.join(__dir__, '.build', 'logs'),
derived_data_path: File.join(__dir__, '.build', 'derived-data')
derived_data_path: File.join(__dir__, '.build', 'derived-data'),
output_style: 'raw'
Copy link
Contributor

Choose a reason for hiding this comment

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

This change was for debugging, wasn't it? It should be okay to delete it now?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes. Great catch, thank you! 🙇‍♂️

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Addressed by dropping the commit via interactive rebase 👌

@mokagio mokagio merged commit b3bcb7f into trunk Aug 9, 2022
@mokagio mokagio deleted the test-analytics branch August 9, 2022 04:32
AliSoftware added a commit that referenced this pull request Aug 9, 2022
With the newer Xcode CI image introduced in #654, we need to install an even newer version of bundler to work around Automattic/a8c-ci-toolkit-buildkite-plugin#16.

This was fixed by #mokagio in various places in the `pipeline.yml` back when he worked on #654, but the `publish-pod.sh` file was missed in this update of the workaround, so we're patching this here
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants