Skip to content

Conversation

rwdougla
Copy link
Contributor

Issue #19

Copy link
Member

@vulder vulder left a comment

Choose a reason for hiding this comment

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

Thanks for starting the topic.
I mostly fixed some skeleton issue to be up to date with the current HEAD skeleton state. You probably should update you fork at some point, so you can directly start from an up to date skeleton version.
And I added some minor comments.

Otherwise, looks like a good starting point for the topic 👍

rwdougla and others added 2 commits January 18, 2021 16:14
Co-authored-by: Florian Sattler <vuld3r@gmail.com>
Co-authored-by: Florian Sattler <vuld3r@gmail.com>
rwdougla and others added 8 commits January 18, 2021 16:31
Co-authored-by: Florian Sattler <vuld3r@gmail.com>
Co-authored-by: Florian Sattler <vuld3r@gmail.com>
Co-authored-by: Florian Sattler <vuld3r@gmail.com>
Co-authored-by: Florian Sattler <vuld3r@gmail.com>
Co-authored-by: Florian Sattler <vuld3r@gmail.com>
Co-authored-by: Florian Sattler <vuld3r@gmail.com>
Co-authored-by: Florian Sattler <vuld3r@gmail.com>
@vulder vulder added the Tentatively ready Ready for a final review in SG20 label Mar 7, 2021
@vulder vulder requested a review from cjdb March 7, 2021 21:49
Copy link
Collaborator

@cjdb cjdb left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for drafting this!

@vulder vulder merged commit e657d2f into cplusplus:master Mar 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Tentatively ready Ready for a final review in SG20
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants