-
Notifications
You must be signed in to change notification settings - Fork 24
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
AttributeError
in new for loop checking
#19
AttributeError
in new for loop checking
#19
Comments
@dopplershift thanks, I was not expecting this one :-) I will try to fix it as soon as I can, pull requests are welcome, of course. |
We are also getting this after updating to 1.1.0, same traceback (https://ci.ros2.org/view/nightly/job/nightly_linux-aarch64_debug/396/testReport/junit/(root)/rosidl_parser/flake8_xunit_missing_result//) |
For reference, the code being checked that caused our error is here (both subdirectories are checked) and I noticed that there is at least one instance of a tuple in the for loop at https://github.com/ros2/rosidl/blob/774a3cd4509f2d432b7325f60434765a9ccd5cee/rosidl_parser/test/test_parse_primitive_value_string.py#L63 |
Fix (with test) in #20 . |
Fix bug in tuple handling in for loop checking (Fixes #19)
I'm getting this error now that I updated to 1.1.0:
I'm seeing this on Windows, Linux, and Mac on 2.7, 3.5, 3.6. This example code (which pulls a tuple out during iteration) is enough to trigger the error (and removing that line makes the error go away):
The text was updated successfully, but these errors were encountered: