From b1212ed53131329574b9c24dd70ae55c5add6248 Mon Sep 17 00:00:00 2001 From: per1234 Date: Sat, 13 Sep 2025 21:44:31 -0700 Subject: [PATCH 1/2] Specify exact path of dependency license metadata cache in ignore files The "Check Dependencies" templates store a cache of metadata about the licenses of dependencies under the `.licenses` folder. The templates use the default location for this folder, under the root of the repository. Since this folder only contains generated files, it should be excluded from the validation checks performed by unrelated tools. Previously, the markdownlint and Prettier ignore configurations for this folder did not explicitly specify a location in the root of the repository, meaning that it would result in ignoring a folder of that name at any location in the project. In cases like this where a specific path is known, it is best to configure that exact path in order to avoid any chance of unexpected behavior. --- .markdownlintignore | 2 +- .prettierignore | 2 +- workflow-templates/assets/check-markdown/.markdownlintignore | 2 +- .../assets/check-prettier-formatting/.prettierignore | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/.markdownlintignore b/.markdownlintignore index b764cf02..d84302f1 100644 --- a/.markdownlintignore +++ b/.markdownlintignore @@ -1,5 +1,5 @@ # Source: https://github.com/arduino/tooling-project-assets/blob/main/workflow-templates/assets/check-markdown/.markdownlintignore -.licenses/ +/.licenses/ __pycache__/ node_modules/ /other/clang-format-configuration/testdata/golden/samples/ diff --git a/.prettierignore b/.prettierignore index 474b9de0..49da21bf 100644 --- a/.prettierignore +++ b/.prettierignore @@ -1,4 +1,4 @@ -.licenses/ +/.licenses/ __pycache__/ node_modules/ /other/clang-format-configuration/testdata/golden/samples/ diff --git a/workflow-templates/assets/check-markdown/.markdownlintignore b/workflow-templates/assets/check-markdown/.markdownlintignore index 0022748d..6cec4c9e 100644 --- a/workflow-templates/assets/check-markdown/.markdownlintignore +++ b/workflow-templates/assets/check-markdown/.markdownlintignore @@ -1,4 +1,4 @@ # Source: https://github.com/arduino/tooling-project-assets/blob/main/workflow-templates/assets/check-markdown/.markdownlintignore -.licenses/ +/.licenses/ __pycache__/ node_modules/ diff --git a/workflow-templates/assets/check-prettier-formatting/.prettierignore b/workflow-templates/assets/check-prettier-formatting/.prettierignore index b5b48de3..bf6e83d0 100644 --- a/workflow-templates/assets/check-prettier-formatting/.prettierignore +++ b/workflow-templates/assets/check-prettier-formatting/.prettierignore @@ -1,4 +1,4 @@ -.licenses/ +/.licenses/ __pycache__/ node_modules/ poetry.lock From 766bbbb9905e3f5e8680b88d484f6e97b2897421 Mon Sep 17 00:00:00 2001 From: per1234 Date: Mon, 15 Sep 2025 15:45:49 -0700 Subject: [PATCH 2/2] Configure tools to exclude folders containing generated files --- .flake8 | 6 ++++++ .markdownlintignore | 1 + .../assets/check-markdown/.markdownlintignore | 1 + workflow-templates/assets/check-python/.flake8 | 6 ++++++ 4 files changed, 14 insertions(+) diff --git a/.flake8 b/.flake8 index efde3a0c..f1989bff 100644 --- a/.flake8 +++ b/.flake8 @@ -5,6 +5,12 @@ [flake8] doctests = True +exclude = + __pycache__/ + .git/ + /.licenses/ + /site/ + node_modules/ # W503 and W504 are mutually exclusive. PEP 8 recommends line break before. ignore = W503 max-complexity = 10 diff --git a/.markdownlintignore b/.markdownlintignore index d84302f1..51aaee1a 100644 --- a/.markdownlintignore +++ b/.markdownlintignore @@ -1,5 +1,6 @@ # Source: https://github.com/arduino/tooling-project-assets/blob/main/workflow-templates/assets/check-markdown/.markdownlintignore /.licenses/ +/site/ __pycache__/ node_modules/ /other/clang-format-configuration/testdata/golden/samples/ diff --git a/workflow-templates/assets/check-markdown/.markdownlintignore b/workflow-templates/assets/check-markdown/.markdownlintignore index 6cec4c9e..c295f6c9 100644 --- a/workflow-templates/assets/check-markdown/.markdownlintignore +++ b/workflow-templates/assets/check-markdown/.markdownlintignore @@ -1,4 +1,5 @@ # Source: https://github.com/arduino/tooling-project-assets/blob/main/workflow-templates/assets/check-markdown/.markdownlintignore /.licenses/ +/site/ __pycache__/ node_modules/ diff --git a/workflow-templates/assets/check-python/.flake8 b/workflow-templates/assets/check-python/.flake8 index efde3a0c..f1989bff 100644 --- a/workflow-templates/assets/check-python/.flake8 +++ b/workflow-templates/assets/check-python/.flake8 @@ -5,6 +5,12 @@ [flake8] doctests = True +exclude = + __pycache__/ + .git/ + /.licenses/ + /site/ + node_modules/ # W503 and W504 are mutually exclusive. PEP 8 recommends line break before. ignore = W503 max-complexity = 10