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

Added export to Joystick::Identification #811

Merged
merged 1 commit into from Feb 28, 2015

Conversation

Projects
None yet
4 participants
@mantognini
Member

mantognini commented Feb 28, 2015

Proper fix for #809.

NB: just testing the web based editor to branch/edit/PR... seems to be working nicely!

@binary1248

This comment has been minimized.

Show comment
Hide comment
@binary1248

binary1248 Feb 28, 2015

Member

Since this is such a trivial change... 👍

Let's see how long the merge takes 😛.

Member

binary1248 commented Feb 28, 2015

Since this is such a trivial change... 👍

Let's see how long the merge takes 😛.

@eXpl0it3r

This comment has been minimized.

Show comment
Hide comment
@eXpl0it3r

eXpl0it3r Feb 28, 2015

Member

This PR has been added to my merge list, meaning it will be merged soon, unless someone raises any concerns.

Member

eXpl0it3r commented Feb 28, 2015

This PR has been added to my merge list, meaning it will be merged soon, unless someone raises any concerns.

@eXpl0it3r eXpl0it3r merged commit 67c7663 into master Feb 28, 2015

@eXpl0it3r eXpl0it3r deleted the export-joy-id branch Feb 28, 2015

@eXpl0it3r

This comment has been minimized.

Show comment
Hide comment
@eXpl0it3r

eXpl0it3r Feb 28, 2015

Member

@binary1248 ~4h and even with a "non-standard" branch name... 😉

Member

eXpl0it3r commented Feb 28, 2015

@binary1248 ~4h and even with a "non-standard" branch name... 😉

@binary1248

This comment has been minimized.

Show comment
Hide comment
@binary1248

binary1248 Feb 28, 2015

Member

+points for being able to cope with the branch name... I wouldn't have been able to get that done in so little time given the circumstances 😆.

Member

binary1248 commented Feb 28, 2015

+points for being able to cope with the branch name... I wouldn't have been able to get that done in so little time given the circumstances 😆.

@MarioLiebisch

This comment has been minimized.

Show comment
Hide comment
@MarioLiebisch

MarioLiebisch Feb 28, 2015

Member

Yeah, that's lame, think of all the other branches that got tidy and use a clean branch name. They have to wait in line while the rowdy rushes through. :P

Member

MarioLiebisch commented Feb 28, 2015

Yeah, that's lame, think of all the other branches that got tidy and use a clean branch name. They have to wait in line while the rowdy rushes through. :P

@mantognini

This comment has been minimized.

Show comment
Hide comment
@mantognini

mantognini Feb 28, 2015

Member

If the name was standard there would not have been any challenge! 😜

Member

mantognini commented Feb 28, 2015

If the name was standard there would not have been any challenge! 😜

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment