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

[Feature] Print functionality #1927

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

joshuaebert
Copy link

Hey,
this PR introduces a Print operation. It's able to print any text (including registers) to the output.

Kind regards
Joshua

@CLAassistant
Copy link

CLAassistant commented Oct 19, 2024

CLA assistant check
All committers have signed the CLA.

@a3957273
Copy link
Member

Hey @joshuaebert, thanks so much for the contribution. A couple of things:

  • Could you add some tests to this operation? See some other PRs for how you can do that.
  • Could you give me an example of a use of this operation? I think I'm failing to see how this is different to a 'concatenate' operator, or with the replace operator how this is different to just starting a new input? You're presumably losing all history from the flow.

@joshuaebert
Copy link
Author

Hey @a3957273,
I'll add some tests once I find the time. Totally forgot about them. Sorry about that 😄

My motivation behind the print operation came from an assignment one of my professors gave my class to create an AES encryption/decryption workflow using cyberchef.
The naive solution was to split encryption and decryption into two recipes, manually copying the output of the encryption recipe to the input of the decryption recipe. My goal was to allow encryption and decryption in a single recipe which would be possible using the print operation I quickly implemented. I can provide you with a screenshot once I'm home.
If theres already the possibility to do that which I missed then I would love to know how. 😄

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

Successfully merging this pull request may close these issues.

3 participants