Add function to show public holiday result

[KevCui]

[sudocanttype]

Nice. Can you write some test cases for it and send the testoutputs.txt? Or if you don't want to, I can make a new PR with the testcases.

[sudocanttype]

K I made a pr to your main for testing. @KevCui

[KevCui]

Thanks @sudocanttype 👍

[sudocanttype]

output.txt Proof that it works. If someone else wants to test before we merge in, go for it

[sudocanttype]

@BeyondMagic Can you test this branch and see if it works for you? If it does, i think this is ready to merge

[BeyondMagic]

I think Basic Answers has conflict with Public Holidays, Public Holidays should be put before it.

[sudocanttype]

Can you give me an example of where it conflicts? I need something to test to fix the problem

[BeyondMagic]

try "christmas" with both features

[sudocanttype]

oh yea, i see it

[BeyondMagic]

also, I think we can add a better sed there, like

holiday's name
* holiday's day
holiday's name
* holiday's day
...

or

holiday's name - holiday's day
holiday's name - holiday's day
holiday's name - holiday's day
...

[sudocanttype]

what exactly is ./tuxi christmas supposed to return? are we trying to give the definition?

[BeyondMagic]

I mean, it should, but it's not.

[sudocanttype]

This doesnt seem to be a problem with the public holiday function. Try checking out the main branch of tuxi and trying ./tuxi christmas. For me it still returns the same thing, so something else is messing with it

[BeyondMagic]

Oh yeah, I remember fixing this in develop branch, I thought this would fix this too in the main since the name is for public holidays

[sudocanttype]

How did you fix it in the develop branch?

[BeyondMagic]

by adding -a to see all results, lol