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 Aruba #780
Update Aruba #780
Conversation
…he difference between 0.8.8 and 0.9 is miniscule in terms of code but it is much easier to then use 0.9.10 to compile Alternatively, we could just set this to be >= 0.8.8 < 1.0??
The implementation of check_file_content was changed to only do an exact match when the second argument is a string, and subsequently the method was deprecated. This fixes both problems in one go.
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.
If anyone knows Aruba, it's mvz.
I think this is ready when green.
I am going to squash-and-merge this to master now, and I'd like to thank @mvz for painstakingly doing this work. |
Hm, maybe we should check the build failure first? |
(And I hate squash merging 🤷♂️) |
It's the only available allowed PR merge option configured in this repository. |
OK, trying again, now |
All green! 🍏 |
This is a based on #772 and fixes the remaining deprecation warnings and test failures.