Remove lint_roller.sh script. It's served its purpose.

Author: domdfcoding

repo_helper/files/linting.py

@@ -42,6 +42,7 @@
 		"lint_warn_list",
 		"make_pylintrc",
 		"make_lint_roller",
+		"remove_lint_roller",
 		"code_only_warning",
 		]
 
@@ -293,6 +294,22 @@ def make_pylintrc(repo_path: pathlib.Path, templates: jinja2.Environment) -> Lis
 
 
 @management.register("lint_roller")
+def remove_lint_roller(repo_path: pathlib.Path, templates: jinja2.Environment) -> List[str]:
+	"""
+	Remove the old lint_roller.sh script to the desired repo.
+
+	:param repo_path: Path to the repository root.
+	:param templates:
+	"""
+
+	lint_file = PathPlus(repo_path / "lint_roller.sh")
+	if lint_file.is_file():
+		lint_file.unlink()
+
+	return [lint_file.name]
+
+
+# @management.register("lint_roller")
 def make_lint_roller(repo_path: pathlib.Path, templates: jinja2.Environment) -> List[str]:
 	"""
 	Add the lint_roller.sh script to the desired repo.

tests/test_core_/test_via_run_repo_helper_forward_stdout.txt

@@ -32,7 +32,6 @@ The following files will be committed:
   MANIFEST.in
   README.rst
   __pkginfo__.py
-  lint_roller.sh
   pyproject.toml
   repo_helper.yml
   setup.cfg

tests/test_files/test_linting.py

@@ -27,7 +27,7 @@
 from pytest_regressions.file_regression import FileRegressionFixture
 
 # this package
-from repo_helper.files.linting import make_lint_roller, make_pylintrc
+from repo_helper.files.linting import make_lint_roller, make_pylintrc, remove_lint_roller
 from tests.common import check_file_output
 
 
@@ -37,6 +37,13 @@ def test_pylintrc(tmp_pathplus, demo_environment, file_regression: FileRegressio
 	check_file_output(tmp_pathplus / managed_files[0], file_regression)
 
 
+def test_lint_roller_removal(tmp_pathplus, demo_environment):
+	managed_files = remove_lint_roller(tmp_pathplus, demo_environment)
+	assert managed_files == ["lint_roller.sh"]
+	assert not (tmp_pathplus / managed_files[0]).exists()
+	assert not (tmp_pathplus / managed_files[0]).is_file()
+
+
 def test_lint_roller_case_1(tmp_pathplus, demo_environment, file_regression: FileRegressionFixture):
 	managed_files = make_lint_roller(tmp_pathplus, demo_environment)
 	assert managed_files == ["lint_roller.sh"]