Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

arduino-lint 1.2.1 requirements fix prevents upload with arduino-cli #1769

Merged
merged 2 commits into from
Jul 21, 2022

Conversation

fpistm
Copy link
Member

@fpistm fpistm commented Jul 21, 2022

Currently, defining a default upload tools prevents flashing with arduino-cli as default methods is always used even if new one is passed as fqbn parameters.

See:
#1654
arduino/arduino-cli#1444

So force usage of 1.2.0 release of the arduino-lint.

This reverts partially commit a4e4377.

Currently, defining a default upload tools prevents flashing
with arduino-cli as default methods is always used even if new
one is passed as fqbn parameters.

See:
stm32duino#1654
arduino/arduino-cli#1444

Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
@fpistm fpistm added bug 🐛 Something isn't working fix 🩹 Bug fix labels Jul 21, 2022
@fpistm fpistm added this to the 2.3.1/2.4.0 milestone Jul 21, 2022
@fpistm fpistm changed the title arduino-lint 1.20.1 requirements fix prevents upload with arduino-cli arduino-lint 1.2.1 requirements fix prevents upload with arduino-cli Jul 21, 2022
new version 1.2.1 raised an error about default upload method.
But it could not be fixed easily as Arduino seems not manage
submenu level for upload method.

See:
    stm32duino#1654
    arduino/arduino-cli#1444

Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
@fpistm fpistm force-pushed the arduino-lint-fix branch from fcc7217 to 54269fe Compare July 21, 2022 09:16
@fpistm fpistm merged commit 46297e2 into stm32duino:main Jul 21, 2022
@fpistm fpistm deleted the arduino-lint-fix branch July 21, 2022 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something isn't working fix 🩹 Bug fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant