Skip to content
This repository was archived by the owner on Oct 24, 2022. It is now read-only.

Conversation

thiblahute
Copy link
Contributor

No description provided.

Copy link
Contributor

@sdroege sdroege left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good otherwise, thanks!

@sdroege
Copy link
Contributor

sdroege commented Aug 16, 2022

Also best to wait for gtk-rs/gtk-rs-core#734 (comment)

@MathieuDuponchelle
Copy link
Collaborator

@sdroege your comment is addressed here right?

@sdroege
Copy link
Contributor

sdroege commented Aug 17, 2022

No

@MathieuDuponchelle
Copy link
Collaborator

ah my bad

@thiblahute
Copy link
Contributor Author

thiblahute commented Aug 17, 2022

Also best to wait for gtk-rs/gtk-rs-core#734 (comment)

This hasn't been done yet afaiu. Other comments should be addressed

@MathieuDuponchelle
Copy link
Collaborator

This hasn't been done yet afaiu. Other comments should be addressed

OK, please just merge when done then :)

@sdroege
Copy link
Contributor

sdroege commented Aug 17, 2022

gtk-rs/gtk-rs-core#748 is pending, then this can go in

@sdroege
Copy link
Contributor

sdroege commented Aug 17, 2022

That one is merged now

@thiblahute
Copy link
Contributor Author

Updated, should be good to go.

@MathieuDuponchelle MathieuDuponchelle merged commit e0ad7e4 into centricular:main Aug 18, 2022
@MathieuDuponchelle
Copy link
Collaborator

thanks :)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants