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
proposed note on cmake requirements #169
Conversation
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.
I was about to write up a patch based on your comment on #152, but it looks like you beat me to it. :) Just a couple notes, let me know what you think. We should add this note to the README for sure though.
Co-Authored-By: Ryan Curtin <ryan@ratml.org>
It seems I forgot to put the change in HISTORY.md as well. I can create another PR for this, or we can include it in this. Either works. |
I meant to do this during merge but I forgot. I'll open a PR in a second since I already have the modifications handy. |
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.
Oops, forgot to approve it.
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.
Looks good to me as well.
No description provided.