Skip to content

Straw-man for static_assert #62

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 12 commits into from
Mar 12, 2021
Merged

Straw-man for static_assert #62

merged 12 commits into from
Mar 12, 2021

Conversation

rwdougla
Copy link
Contributor

Issue #19

Copy link
Collaborator

@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