@abaplint/core 2.120.0 → 2.120.1

package/build/src/rules/identical_move.js

@@ -54,9 +54,9 @@ class IdenticalMove extends _abap_rule_1.ABAPRule {
             title: "Identical move",
             shortDescription: `Moving the same value from left to right or right to left is redundant.`,
             tags: [_irule_1.RuleTag.SingleFile],
-            badExample: `DATA lv_value TYPE i.
+            badExample: `DATA lv_value TYPE i.
 lv_value = lv_value.`,
-            goodExample: `DATA lv_value TYPE i.
+            goodExample: `DATA lv_value TYPE i.
 lv_value = 5.`,
         };
     }

package/build/src/rules/if_in_if.js

@@ -54,43 +54,43 @@ class IfInIf extends _abap_rule_1.ABAPRule {
             key: "if_in_if",
             title: "IF in IF",
             shortDescription: `Detects nested ifs which can be refactored.`,
-            extendedInformation: `
-Directly nested IFs without ELSE can be refactored to a single condition using AND.
-
-ELSE condtions with directly nested IF refactored to ELSEIF, quickfixes are suggested for this case.
-
-https://docs.abapopenchecks.org/checks/01/
+            extendedInformation: `
+Directly nested IFs without ELSE can be refactored to a single condition using AND.
+
+ELSE condtions with directly nested IF refactored to ELSEIF, quickfixes are suggested for this case.
+
+https://docs.abapopenchecks.org/checks/01/
 https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#keep-the-nesting-depth-low`,
-            badExample: `IF condition1.
-  IF condition2.
-    ...
-  ENDIF.
-ENDIF.
-
-IF condition1.
-  ...
-ELSE.
-  IF condition2.
-    ...
-  ENDIF.
+            badExample: `IF condition1.
+  IF condition2.
+    ...
+  ENDIF.
+ENDIF.
+
+IF condition1.
+  ...
+ELSE.
+  IF condition2.
+    ...
+  ENDIF.
 ENDIF.`,
-            goodExample: `IF ( condition1 ) AND ( condition2 ).
-  ...
-ENDIF.
-
-IF condition1.
-  ...
-ELSEIF condition2.
-  ...
-ENDIF.
-
-CASE variable.
-  WHEN value1.
-  ...
-  WHEN value2.
-    IF condition2.
-      ...
-    ENDIF.
+            goodExample: `IF ( condition1 ) AND ( condition2 ).
+  ...
+ENDIF.
+
+IF condition1.
+  ...
+ELSEIF condition2.
+  ...
+ENDIF.
+
+CASE variable.
+  WHEN value1.
+  ...
+  WHEN value2.
+    IF condition2.
+      ...
+    ENDIF.
 ENDCASE.`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix],
         };

package/build/src/rules/implement_methods.js

@@ -143,9 +143,9 @@ class ImplementMethods extends _abap_rule_1.ABAPRule {
         for (const i of ((_a = file.getStructure()) === null || _a === void 0 ? void 0 : _a.findAllStatements(Statements.ClassImplementation)) || []) {
             const name = (_b = i.findFirstExpression(Expressions.ClassName)) === null || _b === void 0 ? void 0 : _b.getFirstToken().getStr().toUpperCase();
             if (name === impl.identifier.getName().toUpperCase()) {
-                return edit_helper_1.EditHelper.insertAt(file, i.getLastToken().getEnd(), `
-  METHOD ${methodName.toLowerCase()}.
-    RETURN. " todo, implement method
+                return edit_helper_1.EditHelper.insertAt(file, i.getLastToken().getEnd(), `
+  METHOD ${methodName.toLowerCase()}.
+    RETURN. " todo, implement method
   ENDMETHOD.`);
             }
         }

package/build/src/rules/implicit_start_of_selection.js

@@ -55,13 +55,13 @@ class ImplicitStartOfSelection extends _abap_rule_1.ABAPRule {
             key: "implicit_start_of_selection",
             title: "Implicit START-OF-SELECTION",
             shortDescription: `Add explicit selection screen event handling`,
-            extendedInformation: `Only runs for executable programs
-
+            extendedInformation: `Only runs for executable programs
+
 https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-us/abapstart-of-selection.htm`,
             tags: [_irule_1.RuleTag.SingleFile],
-            badExample: `REPORT zfoo.
+            badExample: `REPORT zfoo.
 WRITE 'hello'.`,
-            goodExample: `START-OF-SELECTION.
+            goodExample: `START-OF-SELECTION.
   WRITE 'hello'.`,
         };
     }

package/build/src/rules/in_statement_indentation.js

@@ -64,19 +64,19 @@ class InStatementIndentation extends _abap_rule_1.ABAPRule {
             key: "in_statement_indentation",
             title: "In-statement indentation",
             shortDescription: "Checks alignment within statements which span multiple lines.",
-            extendedInformation: `Lines following the first line should be indented once (2 spaces).
-
-For block declaration statements, lines after the first should be indented an additional time (default: +2 spaces)
+            extendedInformation: `Lines following the first line should be indented once (2 spaces).
+
+For block declaration statements, lines after the first should be indented an additional time (default: +2 spaces)
 to distinguish them better from code within the block.`,
-            badExample: `IF 1 = 1
-  AND 2 = 2.
-  WRITE 'hello' &&
-  'world'.
+            badExample: `IF 1 = 1
+  AND 2 = 2.
+  WRITE 'hello' &&
+  'world'.
 ENDIF.`,
-            goodExample: `IF 1 = 1
-    AND 2 = 2.
-  WRITE 'hello' &&
-    'world'.
+            goodExample: `IF 1 = 1
+    AND 2 = 2.
+  WRITE 'hello' &&
+    'world'.
 ENDIF.`,
             tags: [_irule_1.RuleTag.Whitespace, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.SingleFile],
         };

package/build/src/rules/indentation.js

@@ -74,23 +74,23 @@ class Indentation extends _abap_rule_1.ABAPRule {
             title: "Indentation",
             shortDescription: `Checks indentation`,
             tags: [_irule_1.RuleTag.Whitespace, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.SingleFile],
-            badExample: `CLASS lcl DEFINITION.
-PRIVATE SECTION.
-METHODS constructor.
-ENDCLASS.
-
-CLASS lcl IMPLEMENTATION.
-METHOD constructor.
-ENDMETHOD.
+            badExample: `CLASS lcl DEFINITION.
+PRIVATE SECTION.
+METHODS constructor.
+ENDCLASS.
+
+CLASS lcl IMPLEMENTATION.
+METHOD constructor.
+ENDMETHOD.
 ENDCLASS.`,
-            goodExample: `CLASS lcl DEFINITION.
-  PRIVATE SECTION.
-    METHODS constructor.
-ENDCLASS.
-
-CLASS lcl IMPLEMENTATION.
-  METHOD constructor.
-  ENDMETHOD.
+            goodExample: `CLASS lcl DEFINITION.
+  PRIVATE SECTION.
+    METHODS constructor.
+ENDCLASS.
+
+CLASS lcl IMPLEMENTATION.
+  METHOD constructor.
+  ENDMETHOD.
 ENDCLASS.`,
         };
     }

package/build/src/rules/index.js

@@ -45,6 +45,7 @@ __exportStar(require("./check_subrc"), exports);
 __exportStar(require("./check_syntax"), exports);
 __exportStar(require("./check_text_elements"), exports);
 __exportStar(require("./check_transformation_exists"), exports);
+__exportStar(require("./clear_exporting_parameters"), exports);
 __exportStar(require("./class_attribute_names"), exports);
 __exportStar(require("./classic_exceptions_overlap"), exports);
 __exportStar(require("./cloud_types"), exports);

package/build/src/rules/intf_referencing_clas.js

@@ -26,9 +26,9 @@ class IntfReferencingClas {
             key: "intf_referencing_clas",
             title: "INTF referencing CLAS",
             shortDescription: `Interface contains references to class`,
-            extendedInformation: `Only global interfaces are checked.
-      Only first level references are checked.
-      Exception class references are ignored.
+            extendedInformation: `Only global interfaces are checked.
+      Only first level references are checked.
+      Exception class references are ignored.
       Void references are ignored.`,
         };
     }

package/build/src/rules/invalid_table_index.js

@@ -55,9 +55,9 @@ class InvalidTableIndex extends _abap_rule_1.ABAPRule {
             title: "Invalid Table Index",
             shortDescription: `Issues error for constant table index zero, as ABAP starts from 1`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix],
-            badExample: `DATA(first) = table[ 0 ].
+            badExample: `DATA(first) = table[ 0 ].
 READ TABLE gt_stack ASSIGNING <ls_stack> INDEX 0.`,
-            goodExample: `DATA(first) = table[ 1 ].
+            goodExample: `DATA(first) = table[ 1 ].
 READ TABLE gt_stack ASSIGNING <ls_stack> INDEX 1.`,
         };
     }

package/build/src/rules/line_break_style.js

@@ -15,8 +15,8 @@ class LineBreakStyle {
         return {
             key: "line_break_style",
             title: "Makes sure line breaks are consistent in the ABAP code",
-            shortDescription: `Enforces LF as newlines in ABAP files
-
+            shortDescription: `Enforces LF as newlines in ABAP files
+
 abapGit does not work with CRLF`,
             tags: [_irule_1.RuleTag.Whitespace, _irule_1.RuleTag.SingleFile],
         };

package/build/src/rules/line_length.js

@@ -24,11 +24,11 @@ class LineLength extends _abap_rule_1.ABAPRule {
             key: "line_length",
             title: "Line length",
             shortDescription: `Detects lines exceeding the provided maximum length.`,
-            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#stick-to-a-reasonable-line-length
+            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#stick-to-a-reasonable-line-length
 https://docs.abapopenchecks.org/checks/04/`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
             badExample: "DATA long_text TYPE string VALUE 'this line keeps going until it is longer than the configured maximum'.",
-            goodExample: `DATA long_text TYPE string.
+            goodExample: `DATA long_text TYPE string.
 long_text = 'split long expressions across multiple lines'.`,
         };
     }

package/build/src/rules/line_only_punc.js

@@ -27,7 +27,7 @@ class LineOnlyPunc extends _abap_rule_1.ABAPRule {
             key: "line_only_punc",
             title: "Line containing only punctuation",
             shortDescription: `Detects lines containing only punctuation.`,
-            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#close-brackets-at-line-end
+            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#close-brackets-at-line-end
 https://docs.abapopenchecks.org/checks/16/`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.SingleFile],
             badExample: "zcl_class=>method(\n).",

package/build/src/rules/local_class_naming.js

@@ -30,9 +30,9 @@ class LocalClassNaming extends _abap_rule_1.ABAPRule {
             title: "Local class naming conventions",
             shortDescription: `Allows you to enforce a pattern, such as a prefix, for local class names.`,
             tags: [_irule_1.RuleTag.Naming, _irule_1.RuleTag.SingleFile],
-            badExample: `CLASS helper DEFINITION.
+            badExample: `CLASS helper DEFINITION.
 ENDCLASS.`,
-            goodExample: `CLASS lcl_helper DEFINITION.
+            goodExample: `CLASS lcl_helper DEFINITION.
 ENDCLASS.`,
         };
     }

package/build/src/rules/local_variable_names.js

@@ -63,15 +63,15 @@ class LocalVariableNames extends _abap_rule_1.ABAPRule {
         return {
             key: "local_variable_names",
             title: "Local variable naming conventions",
-            shortDescription: `
-Allows you to enforce a pattern, such as a prefix, for local variables, constants and field symbols.
+            shortDescription: `
+Allows you to enforce a pattern, such as a prefix, for local variables, constants and field symbols.
 Regexes are case-insensitive.`,
             tags: [_irule_1.RuleTag.Naming, _irule_1.RuleTag.SingleFile],
-            badExample: `FORM bar.
-  DATA foo.
+            badExample: `FORM bar.
+  DATA foo.
 ENDFORM.`,
-            goodExample: `FORM bar.
-  DATA lv_foo.
+            goodExample: `FORM bar.
+  DATA lv_foo.
 ENDFORM.`,
         };
     }

package/build/src/rules/macro_naming.js

@@ -61,9 +61,9 @@ class MacroNaming extends _abap_rule_1.ABAPRule {
             shortDescription: `Allows you to enforce a pattern for macro definitions`,
             extendedInformation: `Use rule "avoid_use" to avoid macros altogether.`,
             tags: [_irule_1.RuleTag.Naming, _irule_1.RuleTag.SingleFile],
-            badExample: `DEFINE something.
+            badExample: `DEFINE something.
 END-OF-DEFINITION.`,
-            goodExample: `DEFINE _something.
+            goodExample: `DEFINE _something.
 END-OF-DEFINITION.`,
         };
     }

package/build/src/rules/main_file_contents.js

@@ -57,10 +57,10 @@ class MainFileContents {
             title: "Main file contents",
             shortDescription: `Checks related to report declarations.`,
             tags: [_irule_1.RuleTag.Syntax],
-            extendedInformation: `Does not run if the target version is Cloud
-
-* PROGs must begin with "REPORT <name>." or "PROGRAM <name>.
-* TYPEs must begin with "TYPE-POOL <name>."
+            extendedInformation: `Does not run if the target version is Cloud
+
+* PROGs must begin with "REPORT <name>." or "PROGRAM <name>.
+* TYPEs must begin with "TYPE-POOL <name>."
 `,
         };
     }

package/build/src/rules/many_parentheses.js

@@ -56,17 +56,17 @@ class ManyParentheses extends _abap_rule_1.ABAPRule {
             title: "Too many parentheses",
             shortDescription: `Searches for expressions where extra parentheses can safely be removed`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix],
-            badExample: `
-IF ( destination IS INITIAL ).
-ENDIF.
-IF foo = boo AND ( bar = lar AND moo = loo ).
-ENDIF.
+            badExample: `
+IF ( destination IS INITIAL ).
+ENDIF.
+IF foo = boo AND ( bar = lar AND moo = loo ).
+ENDIF.
 `,
-            goodExample: `
-IF destination IS INITIAL.
-ENDIF.
-IF foo = boo AND bar = lar AND moo = loo.
-ENDIF.
+            goodExample: `
+IF destination IS INITIAL.
+ENDIF.
+IF foo = boo AND bar = lar AND moo = loo.
+ENDIF.
 `,
         };
     }

package/build/src/rules/max_one_method_parameter_per_line.js

@@ -54,14 +54,14 @@ class MaxOneMethodParameterPerLine extends _abap_rule_1.ABAPRule {
             title: "Max one method parameter definition per line",
             shortDescription: `Keep max one method parameter description per line`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Whitespace],
-            badExample: `
-METHODS apps_scope_token
-  IMPORTING
+            badExample: `
+METHODS apps_scope_token
+  IMPORTING
     body TYPE bodyapps_scope_token client_id TYPE str.`,
-            goodExample: `
-METHODS apps_scope_token
-  IMPORTING
-    body      TYPE bodyapps_scope_token
+            goodExample: `
+METHODS apps_scope_token
+  IMPORTING
+    body      TYPE bodyapps_scope_token
     client_id TYPE str.`,
         };
     }

package/build/src/rules/max_one_statement.js

@@ -21,11 +21,11 @@ class MaxOneStatement extends _abap_rule_1.ABAPRule {
             key: "max_one_statement",
             title: "Max one statement per line",
             shortDescription: `Checks that each line contains only a single statement.`,
-            extendedInformation: `Does not report empty statements, use rule empty_statement for detecting empty statements.
-
-Does not report anything for chained statements.
-
-https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#no-more-than-one-statement-per-line
+            extendedInformation: `Does not report empty statements, use rule empty_statement for detecting empty statements.
+
+Does not report anything for chained statements.
+
+https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#no-more-than-one-statement-per-line
 https://docs.abapopenchecks.org/checks/11/`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.SingleFile],
             badExample: `WRITE foo. WRITE bar.`,

package/build/src/rules/method_length.js

@@ -71,8 +71,8 @@ class MethodLength {
             key: "method_length",
             title: "Method/Form/Function Module Length",
             shortDescription: `Checks relating to method/form/function module length.`,
-            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#keep-methods-small
-
+            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#keep-methods-small
+
 Abstract methods without statements are considered okay.`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
         };

package/build/src/rules/method_overwrites_builtin.js

@@ -19,20 +19,20 @@ class MethodOverwritesBuiltIn extends _abap_rule_1.ABAPRule {
             key: "method_overwrites_builtin",
             title: "Method name overwrites builtin function",
             shortDescription: `Checks Method names that overwrite builtin SAP functions`,
-            extendedInformation: `https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-us/abenbuilt_in_functions_overview.htm
-
-https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#avoid-obscuring-built-in-functions
-
+            extendedInformation: `https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-us/abenbuilt_in_functions_overview.htm
+
+https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#avoid-obscuring-built-in-functions
+
 Interface method names are ignored`,
             tags: [_irule_1.RuleTag.Naming, _irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Styleguide],
-            badExample: `CLASS lcl DEFINITION.
-  PUBLIC SECTION.
-    METHODS matches.
-ENDCLASS.
-
-CLASS lcl IMPLEMENTATION.
-  METHOD matches.
-  ENDMETHOD.
+            badExample: `CLASS lcl DEFINITION.
+  PUBLIC SECTION.
+    METHODS matches.
+ENDCLASS.
+
+CLASS lcl IMPLEMENTATION.
+  METHOD matches.
+  ENDMETHOD.
 ENDCLASS.`,
         };
     }

package/build/src/rules/method_parameter_names.js

@@ -34,11 +34,11 @@ class MethodParameterNames {
             title: "Method parameter naming conventions",
             shortDescription: `Allows you to enforce a pattern, such as a prefix, for method parameter names`,
             tags: [_irule_1.RuleTag.Naming, _irule_1.RuleTag.SingleFile],
-            badExample: `METHODS get_name
-  IMPORTING customer TYPE string
+            badExample: `METHODS get_name
+  IMPORTING customer TYPE string
   RETURNING VALUE(name) TYPE string.`,
-            goodExample: `METHODS get_name
-  IMPORTING iv_customer TYPE string
+            goodExample: `METHODS get_name
+  IMPORTING iv_customer TYPE string
   RETURNING VALUE(rv_name) TYPE string.`,
         };
     }

package/build/src/rules/mix_returning.js

@@ -54,16 +54,16 @@ class MixReturning extends _abap_rule_1.ABAPRule {
             key: "mix_returning",
             title: "Mix of returning and exporting",
             shortDescription: `Checks that methods don't have a mixture of returning and exporting/changing parameters`,
-            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#use-either-returning-or-exporting-or-changing-but-not-a-combination
-
+            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#use-either-returning-or-exporting-or-changing-but-not-a-combination
+
 This syntax is not allowed on versions earlier than 740sp02, https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-us/abennews-740-abap_objects.htm#!ABAP_MODIFICATION_1@1@`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Syntax],
-            badExample: `CLASS lcl DEFINITION.
-  PUBLIC SECTION.
-    METHODS
-      foobar
-        EXPORTING foo TYPE i
-        RETURNING VALUE(rv_string) TYPE string.
+            badExample: `CLASS lcl DEFINITION.
+  PUBLIC SECTION.
+    METHODS
+      foobar
+        EXPORTING foo TYPE i
+        RETURNING VALUE(rv_string) TYPE string.
 ENDCLASS.`,
         };
     }

package/build/src/rules/nesting.js

@@ -57,7 +57,7 @@ class Nesting extends _abap_rule_1.ABAPRule {
             key: "nesting",
             title: "Check nesting depth",
             shortDescription: `Checks for methods exceeding a maximum nesting depth`,
-            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#keep-the-nesting-depth-low
+            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#keep-the-nesting-depth-low
 https://docs.abapopenchecks.org/checks/74/`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
         };

package/build/src/rules/no_aliases.js

@@ -25,12 +25,12 @@ class NoAliases extends _abap_rule_1.ABAPRule {
             title: "No ALIASES",
             shortDescription: `Detects use of the ALIAS statement`,
             tags: [_irule_1.RuleTag.SingleFile],
-            badExample: `INTERFACE lif_foo.
-  METHODS long_method_name.
-  ALIASES short FOR long_method_name.
+            badExample: `INTERFACE lif_foo.
+  METHODS long_method_name.
+  ALIASES short FOR long_method_name.
 ENDINTERFACE.`,
-            goodExample: `INTERFACE lif_foo.
-  METHODS long_method_name.
+            goodExample: `INTERFACE lif_foo.
+  METHODS long_method_name.
 ENDINTERFACE.`,
         };
     }

package/build/src/rules/no_chained_assignment.js

@@ -57,7 +57,7 @@ class NoChainedAssignment extends _abap_rule_1.ABAPRule {
             extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#dont-chain-assignments`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Quickfix],
             badExample: `var1 = var2 = var3.`,
-            goodExample: `var2 = var3.
+            goodExample: `var2 = var3.
 var1 = var2.`,
         };
     }

package/build/src/rules/no_comments_between_methods.js

@@ -55,19 +55,19 @@ class NoCommentsBetweenMethods extends _abap_rule_1.ABAPRule {
             title: "No comments between methods in global classes",
             shortDescription: `It's not possible to have comments between methods in global classes.`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Syntax],
-            badExample: `CLASS zcl_foo IMPLEMENTATION.
-  METHOD first.
-  ENDMETHOD.
-  " comment
-  METHOD second.
-  ENDMETHOD.
+            badExample: `CLASS zcl_foo IMPLEMENTATION.
+  METHOD first.
+  ENDMETHOD.
+  " comment
+  METHOD second.
+  ENDMETHOD.
 ENDCLASS.`,
-            goodExample: `CLASS zcl_foo IMPLEMENTATION.
-  METHOD first.
-  ENDMETHOD.
-
-  METHOD second.
-  ENDMETHOD.
+            goodExample: `CLASS zcl_foo IMPLEMENTATION.
+  METHOD first.
+  ENDMETHOD.
+
+  METHOD second.
+  ENDMETHOD.
 ENDCLASS.`,
         };
     }

package/build/src/rules/no_external_form_calls.js

@@ -53,8 +53,8 @@ class NoExternalFormCalls extends _abap_rule_1.ABAPRule {
             key: "no_external_form_calls",
             title: "No external FORM calls",
             shortDescription: `Detect external form calls`,
-            badExample: `PERFORM foo IN PROGRAM bar.
-
+            badExample: `PERFORM foo IN PROGRAM bar.
+
 PERFORM foo(bar).`,
             tags: [_irule_1.RuleTag.SingleFile],
         };

package/build/src/rules/no_inline_in_optional_branches.js

@@ -54,17 +54,17 @@ class NoInlineInOptionalBranches extends _abap_rule_1.ABAPRule {
             key: "no_inline_in_optional_branches",
             title: "Don't declare inline in optional branches",
             shortDescription: `Don't declare inline in optional branches`,
-            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#dont-declare-inline-in-optional-branches
-
-Considered optional branches:
-* inside IF/ELSEIF/ELSE
-* inside LOOP
-* inside WHILE
-* inside CASE/WHEN, CASE TYPE OF
-* inside DO
-* inside SELECT loops
-
-Not considered optional branches:
+            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#dont-declare-inline-in-optional-branches
+
+Considered optional branches:
+* inside IF/ELSEIF/ELSE
+* inside LOOP
+* inside WHILE
+* inside CASE/WHEN, CASE TYPE OF
+* inside DO
+* inside SELECT loops
+
+Not considered optional branches:
 * TRY/CATCH/CLEANUP`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
         };

package/build/src/rules/no_macros.js

@@ -52,12 +52,12 @@ class NoMacros extends _abap_rule_1.ABAPRule {
             key: "no_macros",
             title: "No macros",
             shortDescription: `Checks that macros are not used`,
-            extendedInformation: `Macros reduce readability and are difficult to debug, use methods or form routines instead.
-
+            extendedInformation: `Macros reduce readability and are difficult to debug, use methods or form routines instead.
+
 https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/abenmacros_guidl.htm`,
             tags: [_irule_1.RuleTag.SingleFile],
-            badExample: `DEFINE _macro.
-  WRITE 'hello'.
+            badExample: `DEFINE _macro.
+  WRITE 'hello'.
 END-OF-DEFINITION.`,
             goodExample: `WRITE 'hello'.`,
         };