Skip to content

Commit 30c2532

Browse files
committed
Use --library-manager=false as default
Previously, the default check mode for library projects was `--library-manager=submit`. The reasoning was this would ensure that no incompatibilities with Library Manager would be introduced during the development phase of libraries. However, the extra checks by default might make the tool less friendly to first time users who already have their library in Library Manager but aren't using the appropriate setting due to not having read the documentation.
1 parent bc4c5ca commit 30c2532

File tree

2 files changed

+9
-4
lines changed

2 files changed

+9
-4
lines changed

internal/configuration/defaults.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ var defaultRuleModes = map[projecttype.Type]map[rulemode.Type]bool{
3939
rulemode.Strict: false,
4040
rulemode.Specification: true,
4141
rulemode.Permissive: false,
42-
rulemode.LibraryManagerSubmission: true,
42+
rulemode.LibraryManagerSubmission: false,
4343
rulemode.LibraryManagerIndexed: false,
4444
rulemode.Official: false,
4545
},

test/test_all.py

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -181,13 +181,18 @@ def test_version(run_command):
181181
def test_arduino_lint_official(run_command):
182182
project_path = test_data_path.joinpath("ARDUINO_LINT_OFFICIAL")
183183

184-
result = run_command(cmd=[project_path])
184+
# Test default to ARDUINO_LINT_OFFICIAL="false"
185+
result = run_command(cmd=["--library-manager", "submit", project_path])
185186
assert not result.ok
186187

187-
result = run_command(cmd=[project_path], custom_env={"ARDUINO_LINT_OFFICIAL": "true"})
188+
result = run_command(
189+
cmd=["--library-manager", "submit", project_path], custom_env={"ARDUINO_LINT_OFFICIAL": "true"}
190+
)
188191
assert result.ok
189192

190-
result = run_command(cmd=[project_path], custom_env={"ARDUINO_LINT_OFFICIAL": "false"})
193+
result = run_command(
194+
cmd=["--library-manager", "submit", project_path], custom_env={"ARDUINO_LINT_OFFICIAL": "false"}
195+
)
191196
assert not result.ok
192197

193198
result = run_command(cmd=[project_path], custom_env={"ARDUINO_LINT_OFFICIAL": "foo"})

0 commit comments

Comments
 (0)