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

Revert e2e test and run spec on Github Actions #385

Merged
merged 4 commits into from
Apr 18, 2020

Conversation

Kesin11
Copy link
Contributor

@Kesin11 Kesin11 commented Apr 4, 2020

fix #383

Currently, the E2E test is not working properly and always failing.
So I'd like to stop using it and instead run the specs that previously run on CircleCI. It will help us to develop xcode-install.
I don't know thy test has not been running in CircleCI recently, but GitHub Actions can be used as an alternative.

I tried to run ci.yml on my forked repository (Kesin11#6) and found some issues.

"fetch" test failed randomly on CI. It works on my machine, but not on GitHub Actions. I commented out it to stabilize the behavior of CI.
And rubcop shows some violations. I just fix it with rubocop --auto-correct

@Kesin11
Copy link
Contributor Author

Kesin11 commented Apr 10, 2020

@milch @jpsim Would you please review it?

@milch
Copy link
Collaborator

milch commented Apr 18, 2020

Looks reasonable, we can investigate why those other tests failed at some point later

@milch milch merged commit 14cec8e into xcpretty:master Apr 18, 2020
@milch
Copy link
Collaborator

milch commented Apr 18, 2020

Thanks for keeping tabs on this! 🚀

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.

Stop triggering E2E test in pull request.
2 participants