Skip to content
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

Add module.processor for all modules #4

Open
dkushche opened this issue May 3, 2022 · 0 comments
Open

Add module.processor for all modules #4

dkushche opened this issue May 3, 2022 · 0 comments
Assignees

Comments

@dkushche
Copy link
Owner

dkushche commented May 3, 2022

For now, not all modules have a defined processor function that could be used for running them without UI. Let's fix it.
Example:
This is good, it's how it should be:

caesar.little_doc = caesar_little_doc
caesar.full_doc = caesar_full_doc
caesar.processor = caesar_processing

This is bad and needs to be fixed:

diffie_hellman.little_doc = diffie_hellman_little_doc
diffie_hellman.full_doc = diffie_hellman_full_doc

Actually, this issue will be fixed after writing the test for all of the modules, but let's fix it now. I already improved modules creator to add processor automatically. So this issue won't happen in the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants