search

Bugswriter / tuxi

Tuxi is a cli assistant. Get answers of your questions instantly.
GNU General Public License v3.0
1.33k stars 74 forks source link

big update, see PR for details #157

Closed PureArtistry closed 3 years ago

PureArtistry commented 3 years ago

sorry for the delay with this, got kinda busy...

completely re-worked processing code, no more stupid lock file and no more crashes added a number of new features and updated help to reflect everything available [ this will fix: #151 #118 #50 implements #137 #87 ]

scrot_20210228-143709

also re-worked test script making it easier to maintain and makes the output easier to parse (more grep friendly) priority order may need fiddling with, needs some testing to figure out best order also the testqueries.txt could do with going through and updating

testoutputs.txt

let me know what you think

Bugswriter commented 3 years ago

I am merging this. can you tell me .. is your version is capable of doing everything which our current tuxi can do? in main?

PureArtistry commented 3 years ago

Ill check the code in main now, give me 10 minutes

PureArtistry commented 3 years ago

the only thing missing is #152 did you want me to add it in and credit or give @sdushantha a chance to send a PR with it in so he gets credited on the blame if he wants to?

PureArtistry commented 3 years ago

I'm going to message @sdushantha and give him a file with the changes made, he can just send that back in a PR

Bugswriter commented 3 years ago

the only thing missing is #152 did you want me to add it in and credit or give @sdushantha a chance to send a PR with it in so he gets credited on the blame if he wants to?

I think. .. if that's the case I should merge your version with main. But we need to do some testing. and make sure it's not going to break.

PureArtistry commented 3 years ago

yeah, as I said the priority might need adjusting, also maybe the default TUXI_DELAY - need more people to test it and see if it holds up

trying to figure out the best way to contact @sdushantha, there isn't an email address and I don't have any social media accounts to contact him/her on the ones they've detailed

I have created an updated tuxi script with the required changes they just have to send it back in a PR

Bugswriter commented 3 years ago

yeah no need to rush. I am also in busy in my exams.. but I like your work. and will merge it to main .. but only when I get sure by other devs. it's working fine.

PureArtistry commented 3 years ago

thanks!

sdushantha commented 3 years ago

the only thing missing is #152 did you want me to add it in and credit or give @sdushantha a chance to send a PR with it in so he gets credited on the blame if he wants to?

@PureArtistry You can go ahead and add it if you want. I am a little busy and don't have much time to work on it

PureArtistry commented 3 years ago

@sdushantha - will do thanks.

Also thanks for your font preview script, never seen convert used for that before - I have since used it in a few of my scripts