revive icon indicating copy to clipboard operation
revive copied to clipboard

Exported: fix check all comment for `var` and `const`

Open mirecl opened this issue 10 months ago • 4 comments

Fix https://github.com/mgechev/revive/issues/1364

mirecl avatar May 21 '25 14:05 mirecl

Couldn't fix the tests :( Does this PR make any sense at all?

mirecl avatar May 21 '25 15:05 mirecl

Is there any test to update to cover the case you raise?

ccoVeille avatar May 21 '25 15:05 ccoVeille

Is there any test to update to cover the case you raise?

Yes, If my PR is useful, I will do everything!

mirecl avatar May 21 '25 15:05 mirecl

Hi @mirecl, thanks for the PR (and issue report).

Is there any test to update to cover the case you raise?

Yes, If my PR is useful, I will do everything!

The PR seems useful but it needs to confirm that by including a test case of the original problem reported at #1365 .

chavacava avatar May 21 '25 15:05 chavacava

https://github.com/mgechev/revive/issues/1364#issuecomment-2995846565

alexandear avatar Jun 23 '25 10:06 alexandear