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.txt #5749

Closed
wants to merge 1 commit into from
Closed

Test.txt #5749

wants to merge 1 commit into from

Conversation

ClarkZhangdi
Copy link

just for test

Thank you for contributing to RxJava. Before pressing the "Create Pull Request" button, please consider the following points:

  • Please give a description about what and why you are contributing, even if it's trivial.

  • Please include the issue list number(s) or other PR numbers in the description if you are contributing in response to those.

  • Please include a reasonable set of unit tests if you contribute new code or change an existing one. If you contribute an operator, (if applicable) please make sure you have tests for working with an empty, just, range of values as well as an error source, with and/or without backpressure and see if unsubscription/cancellation propagates correctly.

just for  test
@ClarkZhangdi
Copy link
Author

yest

@akarnokd
Copy link
Member

Why? Why? Why? Use your own repository for testing.

@akarnokd akarnokd closed this Nov 30, 2017
@artem-zinnatullin
Copy link
Contributor

@akarnokd to make you feel better — I also did this mistake recently (I use GitHub for ~7 years now) uber/okbuck#547 (comment)

I wanted to open PR against my fork to trigger Travis build (it wasn't starting build for 20 minutes so I thought PR might trigger another build).

So I opened my fork on GitHub and clicked on quick button to open PR from recently pushed branch and clicked through ui. Well, turned out that even if you open PR on your fork GitHub still defaults to original repo.

screen shot 2017-11-30 at 10 20 02 am

Just my two satoshis.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants