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

Add source list view and UI for selecting runs to compare. #21

Closed
wants to merge 6 commits into from

Conversation

c9s
Copy link

@c9s c9s commented Apr 6, 2013

Screen Shot 2013-04-06 at 3 52 30 PM

Please take a look.

@c9s
Copy link
Author

c9s commented Apr 6, 2013

I just fixed the mismerge by doing rebase..

The code styling is also fixed.

Please take a look.

@c9s
Copy link
Author

c9s commented Apr 9, 2013

rebased too, please take a look.

@c9s
Copy link
Author

c9s commented Apr 9, 2013

By the way, are you interested in a patch of putting highcharts in xhprof display ?

@billf
Copy link
Contributor

billf commented Apr 9, 2013

highcharts sounds fine as long as its configurable

@c9s
Copy link
Author

c9s commented Apr 9, 2013

cool, I will work for it on another branch.

@c9s
Copy link
Author

c9s commented Apr 10, 2013

is there any concern of this pull request?

@c9s
Copy link
Author

c9s commented Apr 28, 2013

any update here?

dshafik and others added 5 commits October 24, 2013 14:25
 - Add Makefile.local to fix make test's exit status.
mismatched function call and function signature.
Change inline to static inline for C99 compliance with PHP 5.5
@billf
Copy link
Contributor

billf commented Apr 11, 2014

if you're still interested in getting this in, please rebase/resolve/update and sign the contributor agreement at https://code.facebook.com/cla and i'll work with you on landing this.

@c9s
Copy link
Author

c9s commented Apr 11, 2014

Hi, I just signed and rebased the changes, please take a look again.

@guilhermeblanco
Copy link

@epriestley care to review and merge?

@c9s
Copy link
Author

c9s commented Feb 4, 2015

@guilhermeblanco , @epriestley do you want me to rebase the changes?

@LionsAd LionsAd mentioned this pull request Nov 6, 2015
@Minasu Minasu mentioned this pull request Jun 20, 2016
@c9s c9s closed this Feb 12, 2023
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.

5 participants