Skip to content

Sympa::Aliases re-writing with Sympatic.

David Verdin requested to merge github/fork/dverdin/sympa-7.0 into sympa-7.0

Created by: dverdin

OK, Here is where I am about Sympa::liases modules family. I did the following: 1- Adding test files for all modules, 2- Adding them make check-local, 3- Replacing imports with Sympatic when aplicable, 4- Merging modules: got rid of Sympa::Aliases ::CheckMofule. Now an option sets whether aliases need to be used. 5- Removed module guessing from the main Sympa::Aliases module. Modules are explicitely named from calling programs. I know this creates duplicat code in callers and this is bad. But I want modules to be testable without loading Sympa config. that means trying to eradicate contextual behaviour. It is not ficnished and certainly not perfect but I think it is a step. Please let me know if you agree.

Id' like now to remove dependencies to config in order to not call sympa config inside these modules. Only passing them parameters.

I also want to find a good way to choose between modules depending on Sympa config. I still don't know where it should be.

Anyway, comments are welcome!

Merge request reports