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

autopep8 #6

Merged
merged 1 commit into from
Oct 23, 2018
Merged

autopep8 #6

merged 1 commit into from
Oct 23, 2018

Conversation

mlautman
Copy link
Member

Results of running
autopep8 --max-line-length 130 --in-place $1 on each .py file

@mlautman mlautman requested a review from brawner October 16, 2018 22:59
@@ -93,7 +93,7 @@ def message_callback(self, message):
self.timestamps.append(rospy.get_time())

# FIXME: this only works for message of class AnyMsg
#self.sizes.append(len(message._buff))
# self.sizes.append(len(message._buff))
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Commented code

@@ -360,7 +364,7 @@ def set_selected_topics(self, selected_topics):
@type selected_topics: []
"""
rospy.logdebug('set_selected_topics topics={}'.format(
len(selected_topics)))
len(selected_topics)))
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't break line

@mlautman
Copy link
Member Author

@brawner ready for merge?

@brawner
Copy link

brawner commented Oct 22, 2018

@mlautman It's not letting me approve it, perhaps because of the failing test? It looks good to me though

@mlautman mlautman merged commit 90a67ba into master Oct 23, 2018
@mlautman mlautman deleted the pep8 branch October 23, 2018 16:58
mlautman added a commit that referenced this pull request Oct 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants