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

ci: Minor CI improvements #2411

Merged
merged 3 commits into from
Aug 2, 2022
Merged

ci: Minor CI improvements #2411

merged 3 commits into from
Aug 2, 2022

Conversation

calvincestari
Copy link
Member

@calvincestari calvincestari commented Aug 1, 2022

Closes #2410

  • replaces xcpretty with xcbeautify - the xcpretty project looks like it's been abandoned; multiple outstanding issues and PRs with the last update being 4 years ago. This issue highlights the problem in parallelized test runs and xcbeautify is one of the recommended alternatives including being favoured by FastLane. I looked at the project and it seems overall like a good replacement; written in Swift, better performance, and supports parallelized test runs. You can see the xcbeautify output in the CI output for this PR.
  • added a separate step for the JS frontend tests which is only included in the CodegenLib test job.

@netlify
Copy link

netlify bot commented Aug 1, 2022

Deploy Preview for apollo-ios-docs canceled.

Name Link
🔨 Latest commit d2276ec
🔍 Latest deploy log https://app.netlify.com/sites/apollo-ios-docs/deploys/62e831564f98610009b54339

@netlify
Copy link

netlify bot commented Aug 1, 2022

Deploy Preview for apollo-ios-docc canceled.

Name Link
🔨 Latest commit d2276ec
🔍 Latest deploy log https://app.netlify.com/sites/apollo-ios-docc/deploys/62e8315622fe070008eced77

Copy link
Contributor

@AnthonyMDev AnthonyMDev left a comment

Choose a reason for hiding this comment

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

Thank you!

@calvincestari calvincestari merged commit 423f664 into release/1.0 Aug 2, 2022
@calvincestari calvincestari deleted the 1.0/ci-test-jobs branch August 2, 2022 16:12
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.

2 participants