-
Notifications
You must be signed in to change notification settings - Fork 6.7k
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
runners: jlink: Fix NoneType object error #79889
runners: jlink: Fix NoneType object error #79889
Conversation
The commit 221199e presents a bug that makes west flash failed with error. AttributeError: 'NoneType' object has no attribute 'startswith' In function is_tunnel(), tunnel may contain None and has no attribute "startswith". Fix it. Signed-off-by: Phi Bang Nguyen <[email protected]>
attn: @adamdunkels |
looks good! |
CI error seems unrelated to the change:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nite: it feels like it would be clearer if the parameter to is_ip and is_tunnel would be named dev_id. Seems odd to check if a tunnel is a tunnel or if an IP address is an IP address :)
Global CI issue, new PRs has this CI error now. |
Has been fixed in #79904, rebase on main |
The commit 221199e presents a bug that makes west flash failed with error.
AttributeError: 'NoneType' object has no attribute 'startswith'
In function is_tunnel(), tunnel may contain None and has no attribute "startswith".