Skip to content

Poor error reporting for undefined platforms #282

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

Closed
ianfixes opened this issue Feb 15, 2021 · 0 comments · Fixed by #334
Closed

Poor error reporting for undefined platforms #282

ianfixes opened this issue Feb 15, 2021 · 0 comments · Fixed by #334
Labels
bug Something isn't working ci scripts The test runner scripts

Comments

@ianfixes
Copy link
Collaborator

ianfixes commented Feb 15, 2021

via Arduino-CI/action#21

Consider this compilation config

compile:
  platforms:
    - this_doesnt_exist

This produces NoMethodError: undefined method []' for nil:NilClass error` instead of a sensible message about the platform not being defined. This is a regression in the error communication to the user.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ci scripts The test runner scripts
Projects
None yet
1 participant