robotframework-robocop 8.3.2__tar.gz → 8.4.1__tar.gz

robotframework_robocop-8.4.1/.release-please-manifest.json

@@ -0,0 +1,3 @@
+{
+  ".": "8.4.1"
+}

{robotframework_robocop-8.3.2 → robotframework_robocop-8.4.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: robotframework-robocop
-Version: 8.3.2
+Version: 8.4.1
 Summary: Static code analysis tool (linter) and code formatter for Robot Framework
 Project-URL: Bug tracker, https://github.com/MarketSquare/robotframework-robocop/issues
 Project-URL: Source code, https://github.com/MarketSquare/robotframework-robocop

{robotframework_robocop-8.3.2 → robotframework_robocop-8.4.1}/docs/formatter/formatters/SplitTooLongLine.md

@@ -185,13 +185,16 @@ will result in:
     ```robotframework
     *** Keywords ***
     Arguments
-        [Arguments]
-        ...    ${short}
+        [Arguments]    ${short}
         ...    ${veryLongAndJavaLikeArgumentThatWillGoOverAllowedLength}
         ...    ${veryLongAndJavaLikeArgumentThatWillGoOverAllowedLength}
         Step
     ```
 
+    The first argument is kept on the same line as the setting name and the remaining arguments are split one per
+    line. This follows the [Robot Framework style guide](https://docs.robotframework.org/docs/style_guide) and does
+    not conflict with the `first-argument-in-new-line` (SPC18) and `arguments-per-line` (ARG07) linter rules.
+
 === "After (split_on_every_setting_arg set to False)"
 
     ```robotframework

{robotframework_robocop-8.3.2 → robotframework_robocop-8.4.1}/docs/releasenotes/changelog.md

@@ -1,5 +1,19 @@
 # Release notes
 
+## [8.4.1](https://github.com/MarketSquare/robotframework-robocop/compare/v8.4.0...v8.4.1) (2026-07-30)
+
+
+### Bug Fixes
+
+* mark --ignore values as matched for rules below --threshold ([#1786](https://github.com/MarketSquare/robotframework-robocop/issues/1786)) ([70d0ab6](https://github.com/MarketSquare/robotframework-robocop/commit/70d0ab69f42c3b7dbd4f61605929c8122771a52c)), closes [#1775](https://github.com/MarketSquare/robotframework-robocop/issues/1775)
+
+## [8.4.0](https://github.com/MarketSquare/robotframework-robocop/compare/v8.3.2...v8.4.0) (2026-07-30)
+
+
+### Features
+
+* keep first setting argument on the setting line in SplitTooLongLine ([#1784](https://github.com/MarketSquare/robotframework-robocop/issues/1784)) ([59416c8](https://github.com/MarketSquare/robotframework-robocop/commit/59416c86baa8dbe0d5ac0b342bfb06194038e157)), closes [#1723](https://github.com/MarketSquare/robotframework-robocop/issues/1723)
+
 ## [8.3.2](https://github.com/MarketSquare/robotframework-robocop/compare/v8.3.1...v8.3.2) (2026-06-17)
 
 

robotframework_robocop-8.4.1/src/robocop/__init__.py

@@ -0,0 +1 @@
+__version__ = "8.4.1"  # x-release-please-version

{robotframework_robocop-8.3.2 → robotframework_robocop-8.4.1}/src/robocop/formatter/formatters/SplitTooLongLine.py

@@ -90,6 +90,18 @@ class SplitTooLongLine(Formatter):
 
     ```
 
+    Settings such as ``[Arguments]`` and ``[Tags]`` keep their first argument on the same line as the setting
+    name and split the remaining arguments one per line. This follows the Robot Framework style guide and avoids
+    conflicts with the ``first-argument-in-new-line`` (SPC18) and ``arguments-per-line`` (ARG07) linter rules:
+
+    ```robotframework
+    *** Keywords ***
+    Keyword With Multiple Arguments
+        [Arguments]    ${first_argument}
+        ...    ${second_argument}
+        ...    ${third_argument}
+    ```
+
     Supports global formatting params: ``space-count`` and ``separator``.
     """
 
@@ -259,7 +271,13 @@ class SplitTooLongLine(Formatter):
             token_index = 2
         line: list[Token] = list(node.tokens[:token_index])
         tokens, comments = self.split_tokens(
-            node.tokens, line, self.split_on_every_setting_arg, indent, split_types=split_types, keep_types=keep_types
+            node.tokens,
+            line,
+            self.split_on_every_setting_arg,
+            indent,
+            split_types=split_types,
+            keep_types=keep_types,
+            keep_first_on_line=True,
         )
         if indent:
             comments = [Comment([indent, comment, EOL]) for comment in comments]
@@ -294,6 +312,7 @@ class SplitTooLongLine(Formatter):
         indent: Token | int | None = None,
         split_types: set[str] = ARGUMENTS_ONLY,  # type: ignore[assignment]
         keep_types: set[str] | None = None,
+        keep_first_on_line: bool = False,
     ) -> tuple[list[Token], list[Token]]:
         if not keep_types:
             keep_types = set()
@@ -309,6 +328,7 @@ class SplitTooLongLine(Formatter):
         # [COMMENT, SEPARATOR, COMMENT] tokens in the AST, so to preserve the
         # original comment, we need a lookback at the separator tokens.
         last_separator = None
+        first_split_token = True
         for token in tokens:
             if token.type in self.IGNORED_WHITESPACE:
                 continue
@@ -319,7 +339,12 @@ class SplitTooLongLine(Formatter):
             elif token.type in split_types:
                 if token.value == "":
                     token.value = "${EMPTY}"
-                if split_on or not self.col_fit_in_line([*line, separator, token]):
+                # Keep the first argument on the same line as the setting to follow the style guide
+                # (e.g. ``[Arguments]    ${first}``) so the output does not clash with linter rules
+                # first-argument-in-new-line (SPC18) and arguments-per-line (ARG07).
+                keep_on_line = keep_first_on_line and first_split_token
+                first_split_token = False
+                if not keep_on_line and (split_on or not self.col_fit_in_line([*line, separator, token])):
                     if align_new_line and cont_indent is None:  # we are yet to calculate aligned indent
                         cont_indent = Token(Token.SEPARATOR, self.calculate_align_separator(line))
                     line.append(EOL)

{robotframework_robocop-8.3.2 → robotframework_robocop-8.4.1}/src/robocop/runtime/resolver.py

@@ -129,9 +129,12 @@ class RuleMatcher:
     def _is_rule_disabled(self, rule: Rule) -> bool:
         if rule.is_disabled(self.target_version):
             return True
+        # Evaluate --ignore first so the filter is recorded as matched even when the rule is
+        # also filtered out by --threshold, otherwise it is falsely reported as unmatched.
+        ignored = self.ignore_filter.matches(rule)
         if rule.severity < self.threshold and not rule.config.get("severity_threshold"):
             return True
-        return self.ignore_filter.matches(rule)
+        return ignored
 
     def is_rule_fixable(self, rule: Rule) -> bool:
         """Determine if rule is fixable based on --fixable and --unfixable options."""

{robotframework_robocop-8.3.2 → robotframework_robocop-8.4.1}/tests/formatter/formatters/SplitTooLongLine/expected/settings_on_every_arg.robot

@@ -1,129 +1,117 @@
-*** Settings ***
-Documentation    Test documentation for suite with logs of long lines. Test documentation for suite with logs of long lines.
-
-Library    ShortButSpaced
-...    ${1}
-...    ${2}
-...    ${3}
-Library    CustomLibraryWithLongerNameAndSeveralArguments
-...    first_argument
-...    second_argument=${longer_variable_name}
-...    third_argument=${longer_variable_name}
-Library    CustomLibraryWithLongerNameAndSeveralArguments
-...    first_argument
-...    second_argument=${longer_variable_name}
-...    WITH NAME    name
-Library    CustomLibraryWithLongerNameAndSeveralArguments
-...    first_argument
-...    second_argument=${longer_variable_name}
-...    AS    name
-Library
-
-# there is also resource but it rarely goes over, variable imports, suite setup/suite teardown (but indentnested should take care of that)
-Metadata      Name of metadata variable  For more information about *Robot Framework* see http://robotframework.org or visit dedicated documentation site.
-
-Keyword Tags
-...    PS_CSMON_15838
-...    PR_PPD
-...    PR_B450
-...    PR_B650
-...    PR_B850
-...    PR_CS1000
-...    PR_CS1000L
-...    PAR_ECG
-...    PAR_IP_1
-...    PAR_SPO2
-Test Tags
-...    PS_CSMON_15838
-...    PR_PPD
-...    PR_B450
-...    PR_B650
-...    PR_B850
-...    PR_CS1000
-...    PR_CS1000L
-...    PAR_ECG
-...    PAR_IP_1
-...    PAR_SPO2
-Default Tags
-...    PS_CSMON_15838
-...    PR_PPD
-...    PR_B450
-...    PR_B650
-...    PR_B850
-...    PR_CS1000
-...    PR_CS1000L
-...    PAR_ECG
-...    PAR_IP_1
-...    PAR_SPO2
-# comment
-
-
-*** Test Cases ***
-Test with lots of tags
-    [Tags]
-    ...    PS_CSMON_15838
-    ...    PR_PPD
-    ...    PR_B450
-    ...    PR_B650
-    ...    PR_B850
-    ...    PR_CS1000
-    ...    PR_CS1000L
-    ...    PAR_ECG
-    ...    PAR_IP_1
-    ...    PAR_SPO2
-    Prepare
-    Run
-    Assert
-
-Test with comments in settings
-    [Tags]
-    ...    tag
-    ...    tag
-    ...    PS_CSMON_15838
-    ...    PR_PPD
-    ...    PR_B450
-    ...    PR_B650
-    ...    PR_B850
-    ...    PR_CS1000
-    ...    PR_CS1000L
-    ...    PAR_ECG
-    ...    PAR_IP_1
-    ...    PAR_SPO2
-    # comment1
-    # comment2
-    # comment3
-
-
-*** Keywords ***
-Arguments
-    [Arguments]
-    ...    ${short}
-    ...    ${veryLongAndJavaLikeArgumentThatWillGoOverAllowedLength}
-    ...    ${veryLongAndJavaLikeArgumentThatWillGoOverAllowedLength}
-    Step
-
-Arguments multiline
-    [Arguments]    ${short}    ${veryLongAndJavaLikeArgumentThatWillGoOverAllowedLength}
-    ...    ${veryLongAndJavaLikeArgumentThatWillGoOverAllowedLength}
-    Step
-
-Arguments single line over limit
-    [Arguments]
-    ...    ${short}
-    ...    ${veryLongAndJavaLikeArgumentThatWillGoOverAllowedLengthveryLongAndJavaLikeArgumentThatWillGoOverAllowedLength}
-    ...    ${veryLongAndJavaLikeArgumentThatWillGoOverAllowedLength}
-    Step
-
-Comment that goes over the allowed length
-    [Arguments]
-    ...    ${short}
-    ...    ${veryLongAndJavaLikeArgumentThatWillGoOverAllowedLength}
-    # this is long comment and it should be ignored with --skip-comments
-    Step
-
-Invalid arguments
-   [Arguments]    ${Multiline_Argument1}    ${Multiline_Argument2}    ${Multiline_Argument3}    ${Multiline_Argument4}
-                    ${Multiline_Argument5}    ${Multiline_Argument6}    ${Multiline_Argument7}    ${Multiline_Argument8}
-    ...    ${Multiline_Argument9}    ${Multiline_Argument10}    ${Multiline_Argument11}    ${Multiline_Argument12}
-    ...    ${Multiline_Argument13}    ${Multiline_Argument14}    ${Multiline_Argument15}    ${Multiline_Argument16}
-   [Documentation]    Missing continuation line.
+*** Settings ***
+Documentation    Test documentation for suite with logs of long lines. Test documentation for suite with logs of long lines.
+
+Library    ShortButSpaced    ${1}
+...    ${2}
+...    ${3}
+Library    CustomLibraryWithLongerNameAndSeveralArguments    first_argument
+...    second_argument=${longer_variable_name}
+...    third_argument=${longer_variable_name}
+Library    CustomLibraryWithLongerNameAndSeveralArguments    first_argument
+...    second_argument=${longer_variable_name}
+...    WITH NAME    name
+Library    CustomLibraryWithLongerNameAndSeveralArguments    first_argument
+...    second_argument=${longer_variable_name}
+...    AS    name
+Library
+
+# there is also resource but it rarely goes over, variable imports, suite setup/suite teardown (but indentnested should take care of that)
+Metadata      Name of metadata variable  For more information about *Robot Framework* see http://robotframework.org or visit dedicated documentation site.
+
+Keyword Tags    PS_CSMON_15838
+...    PR_PPD
+...    PR_B450
+...    PR_B650
+...    PR_B850
+...    PR_CS1000
+...    PR_CS1000L
+...    PAR_ECG
+...    PAR_IP_1
+...    PAR_SPO2
+Test Tags    PS_CSMON_15838
+...    PR_PPD
+...    PR_B450
+...    PR_B650
+...    PR_B850
+...    PR_CS1000
+...    PR_CS1000L
+...    PAR_ECG
+...    PAR_IP_1
+...    PAR_SPO2
+Default Tags    PS_CSMON_15838
+...    PR_PPD
+...    PR_B450
+...    PR_B650
+...    PR_B850
+...    PR_CS1000
+...    PR_CS1000L
+...    PAR_ECG
+...    PAR_IP_1
+...    PAR_SPO2
+# comment
+
+
+*** Test Cases ***
+Test with lots of tags
+    [Tags]    PS_CSMON_15838
+    ...    PR_PPD
+    ...    PR_B450
+    ...    PR_B650
+    ...    PR_B850
+    ...    PR_CS1000
+    ...    PR_CS1000L
+    ...    PAR_ECG
+    ...    PAR_IP_1
+    ...    PAR_SPO2
+    Prepare
+    Run
+    Assert
+
+Test with comments in settings
+    [Tags]    tag
+    ...    tag
+    ...    PS_CSMON_15838
+    ...    PR_PPD
+    ...    PR_B450
+    ...    PR_B650
+    ...    PR_B850
+    ...    PR_CS1000
+    ...    PR_CS1000L
+    ...    PAR_ECG
+    ...    PAR_IP_1
+    ...    PAR_SPO2
+    # comment1
+    # comment2
+    # comment3
+
+
+*** Keywords ***
+Arguments
+    [Arguments]    ${short}
+    ...    ${veryLongAndJavaLikeArgumentThatWillGoOverAllowedLength}
+    ...    ${veryLongAndJavaLikeArgumentThatWillGoOverAllowedLength}
+    Step
+
+Arguments multiline
+    [Arguments]    ${short}    ${veryLongAndJavaLikeArgumentThatWillGoOverAllowedLength}
+    ...    ${veryLongAndJavaLikeArgumentThatWillGoOverAllowedLength}
+    Step
+
+Arguments single line over limit
+    [Arguments]    ${short}
+    ...    ${veryLongAndJavaLikeArgumentThatWillGoOverAllowedLengthveryLongAndJavaLikeArgumentThatWillGoOverAllowedLength}
+    ...    ${veryLongAndJavaLikeArgumentThatWillGoOverAllowedLength}
+    Step
+
+Comment that goes over the allowed length
+    [Arguments]    ${short}
+    ...    ${veryLongAndJavaLikeArgumentThatWillGoOverAllowedLength}
+    # this is long comment and it should be ignored with --skip-comments
+    Step
+
+Invalid arguments
+   [Arguments]    ${Multiline_Argument1}    ${Multiline_Argument2}    ${Multiline_Argument3}    ${Multiline_Argument4}
+                    ${Multiline_Argument5}    ${Multiline_Argument6}    ${Multiline_Argument7}    ${Multiline_Argument8}
+    ...    ${Multiline_Argument9}    ${Multiline_Argument10}    ${Multiline_Argument11}    ${Multiline_Argument12}
+    ...    ${Multiline_Argument13}    ${Multiline_Argument14}    ${Multiline_Argument15}    ${Multiline_Argument16}
+   [Documentation]    Missing continuation line.

{robotframework_robocop-8.3.2 → robotframework_robocop-8.4.1}/tests/formatter/formatters/SplitTooLongLine/expected/settings_skip_tests.robot

@@ -30,8 +30,7 @@ Test with comments in settings
 
 *** Keywords ***
 Arguments
-    [Arguments]
-    ...    ${short}
+    [Arguments]    ${short}
     ...    ${veryLongAndJavaLikeArgumentThatWillGoOverAllowedLength}
     ...    ${veryLongAndJavaLikeArgumentThatWillGoOverAllowedLength}
     Step
@@ -42,15 +41,13 @@ Arguments multiline
     Step
 
 Arguments single line over limit
-    [Arguments]
-    ...    ${short}
+    [Arguments]    ${short}
     ...    ${veryLongAndJavaLikeArgumentThatWillGoOverAllowedLengthveryLongAndJavaLikeArgumentThatWillGoOverAllowedLength}
     ...    ${veryLongAndJavaLikeArgumentThatWillGoOverAllowedLength}
     Step
 
 Comment that goes over the allowed length
-    [Arguments]
-    ...    ${short}
+    [Arguments]    ${short}
     ...    ${veryLongAndJavaLikeArgumentThatWillGoOverAllowedLength}
     # this is long comment and it should be ignored with --skip-comments
     Step

{robotframework_robocop-8.3.2 → robotframework_robocop-8.4.1}/tests/linter/test_rule_matcher.py

@@ -455,3 +455,21 @@ class TestRuleMatcher:
         rule_matcher.check_unmatched_filters()
         _, err = capsys.readouterr()
         assert err == ""
+
+    def test_check_matched_ignored_rule_below_threshold(self, capsys):
+        """Rule filtered out by --threshold should still mark --ignore as matched (#1775)."""
+        rule_matcher = RuleMatcher(
+            select=[],
+            extend_select=[],
+            ignore=["some-message-0101"],
+            target_version=ROBOT_VERSION,
+            threshold=RuleSeverity.ERROR,
+            fixable=[],
+            unfixable=[],
+        )
+
+        assert not rule_matcher.is_rule_enabled(get_enabled_rule("0101", severity=RuleSeverity.WARNING))
+
+        rule_matcher.check_unmatched_filters()
+        _, err = capsys.readouterr()
+        assert err == ""

robotframework_robocop-8.3.2/.release-please-manifest.json

@@ -1,3 +0,0 @@
-{
-  ".": "8.3.2"
-}

robotframework_robocop-8.3.2/src/robocop/__init__.py

@@ -1 +0,0 @@
-__version__ = "8.3.2"  # x-release-please-version