Skip to content

Issue 14 #16

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
May 28, 2021
Merged

Issue 14 #16

merged 3 commits into from
May 28, 2021

Conversation

karnkaul
Copy link
Member

Close #14

@karnkaul karnkaul requested a review from hentai-chan May 28, 2021 01:09
@karnkaul
Copy link
Member Author

@hentai-chan Keeping this as a draft; your discretion on whether you'd want to merge it into issue-9 first (just make it a full PR), or after that's been merged into master (close the draft PR, I'll rebase this branch once issue-9 has been merged).

- Emit all compile warnings as errors except on Debug.
@karnkaul karnkaul marked this pull request as ready for review May 28, 2021 06:32
@hentai-chan
Copy link
Contributor

Because this PR fixes some issues of my PR you can go ahead target the issue-9 branch!

@karnkaul
Copy link
Member Author

Because this PR fixes some issues of my PR you can go ahead target the issue-9 branch!

Yup, that's what it should already be targeting!

@hentai-chan hentai-chan merged commit 46dc806 into issue-9 May 28, 2021
@hentai-chan hentai-chan deleted the issue-14 branch May 28, 2021 06:57
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