search

herth / ltk

42 stars 13 forks source link

Added support for ABCL #17

Closed alejandrozf closed 2 years ago

alejandrozf commented 2 years ago

I noticed that LTK didn't worked with ABCL so I inspected the code and noticed that ABCL backend was not implemented in ltk:do-execute and added support for it. I tested with a few examples and it worked fine

herth commented 2 years ago

Hi Alejandro,

thanks for your patch! I am happy to add ABCL support to ltk. It is nice to see, that it only required a few lines of code to achieve this. I have however not merged your pull request, as it contains a ton of whitespace changes. While those should be probably cleaned up in the LTk code base, I don't think this should be done on a commit which also adds functionality. So I added your patch manually and pushed it. I will try in a separate step to clean up the white spaces a bit. You should find the change on the master branch now.

Peter

alejandrozf commented 2 years ago

Hi Peter,

Glad to know you liked the change and now is part of your project so that ABCL is now supported too :)