Deprecate Robotidy for Robocop 6.0+ (#743)

* Deprecate Robotidy

* Deprecate Robotidy for Robocop 6.0+

* Remove dependabot

Author: bhirsz

.github/dependabot.yml

@@ -16,6 +16,12 @@ Robotidy
 - [Usage](#usage)
 - [Example](#example)
 
+DEPRECATION NOTICE
+------------
+Starting from Robocop 6.0, Robotidy is part of Robocop as formatter.
+
+If you are new users, please use https://github.com/MarketSquare/robotframework-robocop instead.
+
 Introduction <a name="introduction"></a>
 ------------
 Robotidy is a tool for autoformatting Robot Framework code.

README.md

@@ -1,3 +1,11 @@
+Deprecation notice
+-------------------
+
+Starting with Robocop 6.0, Robotidy is now integrated into Robocop.
+The standalone version of Robotidy is deprecated and may not support the latest Robot Framework versions.
+
+Please see https://robocop.readthedocs.io/en/stable/ for Robocop documentation.
+
 Introduction
 ------------
 Robotidy is a tool for autoformatting Robot Framework code.

docs/source/overview.rst

@@ -492,6 +492,7 @@ def cli(ctx: click.Context, **kwargs):
     Robotidy is a tool for formatting Robot Framework source code.
     Full documentation available at <https://robotidy.readthedocs.io> .
     """
+    print_deprecation_warning()
     cli_config = RawConfig.from_cli(ctx=ctx, **kwargs)
     global_config = config_module.MainConfig(cli_config)
     global_config.validate_src_is_required()
@@ -507,3 +508,10 @@ def cli(ctx: click.Context, **kwargs):
     tidy = app.Robotidy(global_config)
     status = tidy.transform_files()
     sys.exit(status)
+
+
+def print_deprecation_warning() -> None:
+    print(
+        "DeprecationWarning: Starting with Robocop 6.0, Robotidy is now integrated into Robocop. "
+        "The standalone version of Robotidy is deprecated and may not support the latest Robot Framework versions.\n"
+    )

robotidy/cli.py

@@ -1 +1 @@
-__version__ = "4.16.0"
+__version__ = "4.17.0"

robotidy/version.py

@@ -41,13 +41,13 @@
     include_package_data=True,
     python_requires=">=3.8",
     install_requires=[
-        "robotframework>=4.0,<7.3",
+        "robotframework>=4.0",
         "click==8.1.*",
-        "colorama>=0.4.3,<0.4.7",
-        "pathspec>=0.9.0,<0.12.2",
-        "tomli>=2.0,<2.3",
-        "rich_click>=1.4,<1.8.6",
-        "jinja2>=3.1.3,<4.0",
+        "colorama>=0.4.3",
+        "pathspec>=0.9.0",
+        "tomli>=2.0",
+        "rich_click>=1.4",
+        "jinja2>=3.1.3",
     ],
     extras_require={
         "dev": [

setup.py

@@ -52,7 +52,7 @@ def test_invalid_template_path(self):
             f"Error: {self.TRANSFORMER_NAME}: Invalid 'doc_template' parameter value: '{template_path}'. "
             f"The template path does not exist or cannot be found.\n"
         )
-        assert expected_output == result.output
+        assert expected_output in result.output
 
     def test_invalid_template(self):
         template_path = Path(__file__).parent / "source" / "invalid_template.jinja"
@@ -65,4 +65,4 @@ def test_invalid_template(self):
             f"Failed to load the template: Unexpected end of template. Jinja was looking for the "
             f"following tags: 'endfor' or 'else'. The innermost block that needs to be closed is 'for'.\n"
         )
-        assert expected_output == result.output
+        assert expected_output in result.output

tests/atest/transformers/GenerateDocumentation/test_transformer.py

@@ -69,7 +69,7 @@ def test_invalid_order_parameter(self, parameter):
             "Custom order should be provided in comma separated list with all section names:\n"
             "order=comments,settings,variables,testcases,tasks,variables\n"
         )
-        assert expected_output == result.output
+        assert expected_output in result.output
 
     def test_inline_if(self):
         self.compare(source="inline_if.robot", not_modified=True)

tests/atest/transformers/MergeAndOrderSections/test_merge_and_order_sections.py

@@ -48,7 +48,7 @@ def test_invalid_equal_sign_type(self, param_name):
             f"Error: {self.TRANSFORMER_NAME}: Invalid '=' parameter value: '{param_name}'. "
             "Possible values:\n    remove\n    equal_sign\n    space_and_equal_sign\n"
         )
-        assert expected_output == result.output
+        assert expected_output in result.output
 
     def test_disablers(self):
         self.compare(source="disablers.robot", not_modified=True)

tests/atest/transformers/NormalizeAssignments/test_transformer.py

@@ -32,7 +32,7 @@ def test_wrong_case(self):
             f"Error: {self.TRANSFORMER_NAME}: Invalid 'case' parameter value: 'invalid'. "
             f"Supported cases: lowercase, uppercase, titlecase.\n"
         )
-        assert expected_output == result.output
+        assert expected_output in result.output
 
     def test_only_remove_duplicates(self):
         self.compare(source="duplicates.robot", config=f":normalize_case=False")