lint(utils): mark unused description param in vector_test (fixes #4659)

[omegajudith]

Fixes #4659.

tests/core/pyspec/eth2spec/test/utils/utils.py:

• The vector_test decorator accepted a description argument that was never used.
• Renamed it to _description to indicate intentional non-use and satisfy the linter.

No behavioral changes; only a lint/clarity update.
Ran make lint locally.

[leolara]

Why is also removing the other function?

The goal should be to remove the parameter, but I am also doing it together with other things here: #4658

[omegajudith]

I am still on it . I am fixing it still

[omegajudith]

@leolara please take alook if get a minute

[omegajudith]

@jtraglia , @leolara feed back please. so i can fix

[leolara]

this will create conflict here: #4658 and that ones removes it.

@omegajudith have a look at this one: #4602

i don't think anyone is working on that, and it is self contain. A bit more difficult but will help you learn how the reftests work more.

[omegajudith]

Thank you very much @leolara , let me surely read on tis

[omegajudith]

@leolara you said: >>> this will create conflict here: #4658 and that ones removes it.

i didnt understand this part, >>>> and that ones removes it.

please help clarify. thank you

[leolara]

@leolara you said: >>> this will create conflict here: #4658 and that ones removes it.

i didnt understand this part, >>>> and that ones removes it.

please help clarify. thank you

That if we merged this PR, it will cause a merge conflict with the other one, that removes actually the parameter that is not used, and also does more things like add unit tests.

[omegajudith]

Ah, okay I understand. That was well explained.

So now you say I should stop working on that and start working on issue 4602?

Should I set 4659 back to you upstream? Coz now Iam going to start working on 4602

[leolara]

Yes, so it seems that one is taken, we will come up with another one probably