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

Remove useless manual implementation #571

Merged
merged 1 commit into from Sep 10, 2017

Conversation

Projects
None yet
3 participants
@GuillaumeGomez
Member

GuillaumeGomez commented Sep 3, 2017

@EPashkin

This comment has been minimized.

Show comment
Hide comment
@EPashkin

EPashkin Sep 3, 2017

Member

Hm, duplicate implementation 😉.
Thanks.
👍 after error fixed.

Member

EPashkin commented Sep 3, 2017

Hm, duplicate implementation 😉.
Thanks.
👍 after error fixed.

@EPashkin

This comment has been minimized.

Show comment
Hide comment
@EPashkin

EPashkin Sep 9, 2017

Member

@GuillaumeGomez, May be this too better merged before @sdroege's
Travis just had internal error.

Member

EPashkin commented Sep 9, 2017

@GuillaumeGomez, May be this too better merged before @sdroege's
Travis just had internal error.

@sdroege

This comment has been minimized.

Show comment
Hide comment
@sdroege

sdroege Sep 9, 2017

Member

Doesn't interfere with my PRs, or am I missing something?

Member

sdroege commented Sep 9, 2017

Doesn't interfere with my PRs, or am I missing something?

@EPashkin

This comment has been minimized.

Show comment
Hide comment
@EPashkin

EPashkin Sep 9, 2017

Member

Not much interference

Member

EPashkin commented Sep 9, 2017

Not much interference

@GuillaumeGomez

This comment has been minimized.

Show comment
Hide comment
@GuillaumeGomez

GuillaumeGomez Sep 10, 2017

Member

Then I merge it.

Member

GuillaumeGomez commented Sep 10, 2017

Then I merge it.

@GuillaumeGomez GuillaumeGomez merged commit 28c4c9a into gtk-rs:master Sep 10, 2017

1 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build could not complete due to an error
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment