Andrei Paikin

@vogella sounds interesting. What refactorings are most useful? Is there some guide how to add new JDT cleanup actions? Btw, thank you for your site and tutorials, they were very...

@martin-mfg thanks for your interest in checkstyle. I think you misunderstood issue a little bit. You should not remove existing properties and replace them with new one. It is a...

@martin-mfg It sounds reasonable, and I like this changes. What bothers me is that it is breaking compatibility and issue is quite old, may be there are some nuances. @romani...

Github, generate report

There are cases like this https://checkstyle-diff-reports.s3.us-east-2.amazonaws.com/e5f329e_2024235419/reports/diff/spotbugs/index.html#A1, when annotation is considered as tag. In theory, we can analyse context and ignore text inside `` or `` tags

Problem is in BlockTagUtil#extractBlockTags, it does not take context into consideration. There are 2 possible solutions: 1. Consider tags and and ignore its content 2. Adjust regex to make it...

> Without migration, we just need to make sure that diff report is decently ok or better then before in behavior it seems ok to me if we ignore cases...

@remal instead of `getContents()` checks should use only AST and do not rely on text in file. About the case you described - it seems to me that you dont...

@Kevin222004 I would say it is better to pick up some other issue (with gsoc label), this changed are not easy for new contributors.

> I think it should be a page in https://checkstyle.org/extending.html This is mostly for 3rd-party check/filters/etc., but grammar changes cant be 3rd party, it is a core functionality. We can...