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

simplified formatting #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

abc-mikey
Copy link

There is a lot of unnecessary printing of strings and subshelling going on here. I've simplified it down to gawk being called once with one parameter which holds the calculation and printf is used to convert it to an integer without a new line.

This was referenced Oct 22, 2022
@igo95862
Copy link
Owner

Hello,

What do you think about forking or transferring ownership?

Sorry, I kind of think of this project as feature complete and I don't really feel like spending time maintaining it.

Also I would rewrite in Python if I were to add any new features, anything other than few lines of script in shell is maintenance nightmare.

@m-bdf m-bdf mentioned this pull request Mar 30, 2023
12 tasks
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