Skip to content

Regenerate with gtk 3.99.3 #38

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

Merged
merged 3 commits into from
Nov 5, 2020
Merged

Regenerate with gtk 3.99.3 #38

merged 3 commits into from
Nov 5, 2020

Conversation

bilelmoussaoui
Copy link
Member

Depends on #35

@bilelmoussaoui bilelmoussaoui force-pushed the bilelmoussaoui/update branch 2 times, most recently from ce5014c to ee8df7f Compare November 5, 2020 01:20
@sdroege
Copy link
Member

sdroege commented Nov 5, 2020

You seem to have forgotten to create the gir-files PR :)

Copy link
Member

@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 generally good to me apart from those comments.

@bilelmoussaoui
Copy link
Member Author

You seem to have forgotten to create the gir-files PR :)

This was handled by Chris few days ago already :)

@sdroege
Copy link
Member

sdroege commented Nov 5, 2020

This was handled by Chris few days ago already :)

gtk-rs/gir-files#61 this one? And the bindings were simply not yet updated to that version?

@bilelmoussaoui
Copy link
Member Author

Yes, that's it. There was an sdk update recently though, I will check if there were any changes as I need to add the necessary gir files for #37 as well

@sdroege
Copy link
Member

sdroege commented Nov 5, 2020

Makes sense. Then get this in here once the CI is happy, the changes look good to me :)

same as append_linear_gradient & append_repeating_linear_gradient
@bilelmoussaoui
Copy link
Member Author

This looks good now :)

@bilelmoussaoui bilelmoussaoui merged commit 29a1c7a into master Nov 5, 2020
@bilelmoussaoui bilelmoussaoui deleted the bilelmoussaoui/update branch November 5, 2020 11:46
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.

2 participants