Modified default mutable arguments

[svenkat19]

No description provided.

[chavarera]

@svenkat19 any update?

[svenkat19]

@svenkat19 any update?

Oh sorry, I just forgot to inform, Could you please explain where the doc-string document is?

[chavarera]

@svenkat19 any update?

Oh sorry, I just forgot to inform, Could you please explain where the doc-string document is?

In same code the documentation added please update

[svenkat19]

text_exclude already had a description I suppose!

[chavarera]

s-tool/s_tool/parser.py

Line 10 in 7a5651a

text_exclude : list of values to exclude from result

Here Instead of list put it as immutable object tuple ..

[chavarera]

@svenkat19 after modification of code please run the test case so it will validate your code block properly working or not.

If something breaking please modify accordingly

[svenkat19]

I went through the cchanges, I am not able to find where I am going wrong. Could you help?

[chavarera]

I went through the cchanges, I am not able to find where I am going wrong. Could you help?

@svenkat19

s-tool/s_tool/parser.py

Line 22 in 33f9c03

if text not in text_exclude:

consider if we not passing text_exclude value then above line will generate following exceptions

TypeError: argument of type 'NoneType' is not iterable

before iteration we need to compare the value

if text_exclude and text not in text_exclude:
    #perform regular operation

After completion of code run your code with

pytest

After all testcases passed then push

if you have still doubt comment it

[chavarera]

closing PR

Reason: No update