Jörg-Michael Grassau

Results 139 comments of Jörg-Michael Grassau

Hi, just to be sure that I understand you correctly: So the idea would be to have a generic text replacement rule in which you can enter any search text...

Hi, I must admit I am a bit hesitant: While I see the advantage of having everything done with one keystroke and the flexibility of the idea, this might also...

Hi Edo, that's a good point! The alignment of the CONDITION section should indeed already work with the "Align logical expressions" rule (or the "Move AND/OR etc. from line end...

Hi Vlad, actually the Styleguide is quite clear on that, saying [Align assignments to the same object, **but not to different ones**](https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#align-assignments-to-the-same-object-but-not-to-different-ones), and I have already come across quite a...

Hi Koch013, thanks for opening this – yes, this is also on my own wish list! Things might get tricky in cases like this one: ``` lt_my_tab = VALUE #(...

Hi hansdampfinger666, this is usually handled by the cleanup rule "Align declarations" which, however, is deactivated in your profile. Actually, if the input is … ```ABAP TYPES: BEGIN OF ty_conf_out_posnr....

Hi hansdampfinger666, I now modified the "Indent lines" rule to indent INCLUDE lines like PrettyPrinter would do it. Therefore, with the next ABAP cleaner release, all of the following cases...

Hi hansdampfinger666, thanks again for opening this issue – this should now be fixed (as described above) with version 1.16.2, which was just released. Kind regards, Jörg-Michael

Hi ConjuringCoffee, hm, yes, you're right. Technically, the reason is this: ![image](https://github.com/SAP/abap-cleaner/assets/79639298/29b23261-78f3-4701-a164-faeb1ed5df26) So you get the execution of two rules for the price of one :-) Showing both rules in...

Hi ConjuringCoffee, that's a good point. One possible place for this would be the "Align parameters and components" rule, but I did not want to condense the space before the...