Skip to content

added stdin option to recieve text#3

Open
omarha96 wants to merge 2 commits intoAkronae:mainfrom
omarha96:main
Open

added stdin option to recieve text#3
omarha96 wants to merge 2 commits intoAkronae:mainfrom
omarha96:main

Conversation

@omarha96
Copy link
Copy Markdown

added a stdin option to send the text along with --file, this helps reduce disk read/write when unnecessary.

@omarha96 omarha96 marked this pull request as draft March 23, 2026 21:15
@Akronae
Copy link
Copy Markdown
Owner

Akronae commented Mar 26, 2026

hi @omarha96, thanks, that's definitely an improvement. any reason to downgrade to net472 though?

@omarha96
Copy link
Copy Markdown
Author

omarha96 commented Mar 29, 2026

You're welcome @Akronae . And great solution by the way. I couldn't do it directly in powershell or ps scripts for some reason. This wrapper cli was a life saver.
About the downgrade, it's just my system has an older version and couldn't clean the commit at time, but I think you can just ignore the downgrade change. Sorry about the mess

@Akronae Akronae marked this pull request as ready for review March 29, 2026 12:44
refactor: improve command-line validation for file and stdin options

chore: update project file to target .NET Framework 4.8.1 and include necessary packages
@Akronae
Copy link
Copy Markdown
Owner

Akronae commented Mar 29, 2026

@omarha96 glad it could help! just reverted downgrade and added validation for input mode, if it looks ok with you I can merge that c414395

@omarha96
Copy link
Copy Markdown
Author

omarha96 commented Apr 4, 2026

@Akronae Perfect 👍

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.

2 participants