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
Pull request for MobiPerf v0.1 Beta #23
Conversation
Static reference to account to be used.
Seems still can't be merged. |
…biperf-master Conflicts: .gitignore
OK, now it can be automatically merged after I made a pull request from Main to my local branch and I merge the new updates from the main brach. |
Pull request for MobiPerf v0.1 Beta
Have you tested that this all works when built from exactly these sources? |
I've tested building from these sources, and AFAICT, it is working |
@@ -21,3 +21,4 @@ server/dev_data/ | |||
|
|||
# Random evil files | |||
*.DS_Store | |||
android/*/*/.DS_Store |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Gigantic code review so this is a low priority thing to catch, but I wanted to alert @quietbamboo about this. This is a redundant line. http://linux.die.net/man/5/gitignore
I will fix in my next CL
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You mean "android///.DS_Store" is redundant of "*.DS_Store", right?
I agree. Just remove it.
This pull request includes all previous commits.
Appears a little messy here. Will make modularized pull requests from now.