-
Notifications
You must be signed in to change notification settings - Fork 17
Compatibility with IA-1.0 #178
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
base: master
Are you sure you want to change the base?
Conversation
I had done some work on this as well, mostly through search and replace through the whole repository. I have merge the two version and put it in the The main difference is that:
|
Thanks Benoît for the efforts and the ideas; I like both of them. Currently, I'm now in the "validated" integration part, which I am moving to a submodule (it is sort of a separate thing), so if you want to push your suggestions, please do so. |
(I can also incorporate them, whenever I also find the time.) |
I have added my changes. There are some tests failing that I don't have time to look at right now, but (normally) I fixed all type errors. |
Thanks a lot @Kolaru for including your changes; I'll take some time tomorrow (or the day after) to see what happens with the tests. |
No description provided.