-
Notifications
You must be signed in to change notification settings - Fork 89
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
feat: updated LayoutBuilder.h (added functors) for C++11 compatibility #3281
Conversation
…s is not the final version and may contain errors
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jpivarski - all the tests pass. @HavryliukAY - please, check if it works for you. Thanks!
I tried to contact @HavryliukAY on Mattermost, but I think he's offline. If this PR is not sufficient, we can do another one later to fix any remaining issues. I'll merge it now. |
@all-contributors please add @HavryliukAY for code (I forgot to add you to the contributors because GitHub shows this as my PR. But you did the work!) |
I've put up a pull request to add @HavryliukAY! 🎉 |
@HavryliukAY, since you said,
I'm going to put it in draft mode. When you're ready for a review, click both "Ready for Review" (below tests) and assign me as a reviewer (top-right).