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

Assert check #55

Merged
merged 2 commits into from Jun 8, 2018

Conversation

Projects
None yet
3 participants
@GuillaumeGomez
Member

GuillaumeGomez commented Jun 8, 2018

Fixes #54.

cc @sdroege @EPashkin

@sdroege

This comment has been minimized.

Show comment
Hide comment
@sdroege
Member

sdroege commented Jun 8, 2018

Show outdated Hide outdated src/rt.rs
@sdroege

This comment has been minimized.

Show comment
Hide comment
@sdroege

sdroege Jun 8, 2018

Member

👍 or just move those few lines into lib.rs

Member

sdroege commented Jun 8, 2018

👍 or just move those few lines into lib.rs

@GuillaumeGomez

This comment has been minimized.

Show comment
Hide comment
@GuillaumeGomez

GuillaumeGomez Jun 8, 2018

Member

No, I like having a rt file around. :)

Member

GuillaumeGomez commented Jun 8, 2018

No, I like having a rt file around. :)

@EPashkin

This comment has been minimized.

Show comment
Hide comment
@EPashkin

EPashkin Jun 8, 2018

Member

@GuillaumeGomez Thanks, 👍 too

Member

EPashkin commented Jun 8, 2018

@GuillaumeGomez Thanks, 👍 too

@GuillaumeGomez GuillaumeGomez merged commit fa65dad into gtk-rs:master Jun 8, 2018

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@GuillaumeGomez GuillaumeGomez deleted the GuillaumeGomez:assert-check branch Jun 8, 2018

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