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

test/rdoc/test_rdoc_generator_json_index.rb: pend in Travis ppc64le. #8800

Closed
wants to merge 3 commits into from

Conversation

junaruga
Copy link
Member

@junaruga junaruga commented Oct 30, 2023

This is a workaround for Travis ppc64le test/rdoc/test_rdoc_generator_json_index.rb failing randomly ruby/rdoc#1048. I think pending the test disabling the allow_failures in Travis CI ppc64le is better because we can avoid unintentional new test failures on the case.

After testing it on this repository, I plan to send another PR to ruby/rdoc repository with the 1st commit.

The previous commit for the `test/rdoc/test_rdoc_generator_json_index.rb` is a
workaround to pass Travis ppc64le case by pending the test.
So far I haven't seen any other failures on Travis ppc64le.
I want to see the pending tests for each tests.
It seems that the `--tty=no` hides the pending tests. So, I removed it.
@junaruga
Copy link
Member Author

junaruga commented Nov 3, 2023

I would close this PR, because I merged ruby/rdoc#1050 not depending Travis CI environment variables on the ruby/rdoc repository, and the commit 1f1b9b0 was synchronized from the ruby/rdoc to the ruby/ruby repository.

@junaruga junaruga closed this Nov 3, 2023
@junaruga junaruga deleted the wip/travis-ppc64le-rdoc branch November 3, 2023 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
1 participant