-
Notifications
You must be signed in to change notification settings - Fork 95
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
Update sample projects according to the newest LLVM# API + .Net Core 2.0 #58
Conversation
I also recommend you to make the use of Travis CI. |
My last commit ask Travis CI to build LLVM#. Travis CI will only send you email when you pushed uncompilable code(rather than simply |
@mjsabby @martinwoodward 😄 Please carefully review and merge my pull request |
@ice1000 can you squash merge into a single commit? Also, is there a reason why you removed "this." from your changes? |
It's..ahh just personal reason. I'll revert those changes.
Can you do this as the merger? I've seen the github blog and it says there's a button like this |
All |
@mjsabby 😃 CI check done. Everything's ready |
Thanks @ice1000, merged. |
And fix some stupid errors. 😂