-
Notifications
You must be signed in to change notification settings - Fork 8
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
Framework for Centrallix tests in C #14
Conversation
🎉 All dependencies have been resolved ! |
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 haven't reviewed the actual code yet, but is there a reason that this PR wants to merge into remove_net_raw? Just wanted to ensure that wasn't a goof or an artifact of dpulls or whatnot before I started munching my way through this pull request.
That's just part of chaining pull requests with dpulls - it means that you should see this branch diffed with remove_net_raw, instead of with master. However, as you can see, once remove_new_raw was merged and the branch deleted, it automatically updated to merge into master instead. |
Ah, got it. Dpulls just didn't act quite quickly enough to update this PR. :) |
Actually the issue was that I had to go and manually delete the branch after it was merged, since GitHub doesn't delete it automatically - you still have to click the button! |
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.
Thanks! :)
…ing, detect presence of curses during build
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 - thanks! :)
Depends on #15