-
Notifications
You must be signed in to change notification settings - Fork 42
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
Build fix redo #353
Build fix redo #353
Conversation
Ok, I noticed the windows error message said to try |
Is there anything wrong with me squashing this into a single commit once you have it working? |
Possibly a loss of operational knowledge in the steps one would take to fix these kinds of CI failures? |
I think I could squash them into 1 commit per operating system, with descriptions explaining how I figured out the problems and why I chose the fixes I did. |
Got everything from this cleaned up into #354 |
This is a cleaner redo of the working build fixes from #352. I'd like to get it merged so I can rebase #351 onto it
This stops short of trying to fix the Mac library compatibility warnings, which I haven't figured out how to do yet.
Hopefully the Windows CI failure that was happening last night will go away when the CI runs today (it was a dependency download error that seemed outside of our control.)