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

Ready to enable android bot to make check #10394

Merged
merged 6 commits into from
Nov 11, 2013
Merged

Ready to enable android bot to make check #10394

merged 6 commits into from
Nov 11, 2013

Conversation

yichoi
Copy link
Contributor

@yichoi yichoi commented Nov 10, 2013

To enable test on android bot #9120

some tests are disabled and can be fixed further.

@alexcrichton
Copy link
Member

Do we have a plan for the next steps in enabling continuous tests on android? I have a feeling that as soon as we merge this we're going to start piling up the incompatibilities again. It'd be nice to have a bot turned on (if we need it to be) as this gets landed.

That being said, if it takes awhile to get that up and running, this certainly looks fine as-is and we could merge it.

@yichoi
Copy link
Contributor Author

yichoi commented Nov 10, 2013

I have had experience android breakage many times and fixed urgently to demonstrate from time to time, so I strongly suggest to enable android bot.
I will discuss about this topic with @brson shortly. he will come to Korea soon.

@brson
Copy link
Contributor

brson commented Nov 11, 2013

I'll modify the bots to test Android next week. Sorry I've been taking so long on this.

bors added a commit that referenced this pull request Nov 11, 2013
To enable test on android bot #9120

some tests are disabled and can be fixed further.
@bors bors closed this Nov 11, 2013
@bors bors merged commit 9bb1e25 into rust-lang:master Nov 11, 2013
@yichoi yichoi deleted the make_check_pass_android branch November 19, 2013 04:31
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.

4 participants