-
Notifications
You must be signed in to change notification settings - Fork 1
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
Get to successful compile #14
Conversation
@tonyxforce Does "compile" refer to a successful run of |
Successful run of |
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.
Simple changes. This builds on debian, so the _
should be fine. This project will be heading toward a major refactor, so we'll address the file structure at that point. I made a few comments with regard to some things that can be addressed. Overall, great job. Once the requested changes are made and everything looks good, we'll merge 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.
Consistency with imports syntax and re-enabling deprecation warnings just need to be addressed and this should be good to go 👍
… nonsensetwice-main
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.
Pulled and was able to build. Thank you for this work, @tonyxforce 🔥
tried on windows, haven't on any unix-based OS