search

sugarlabs / showntell-activity

ShowNTell expands the capabilities of ClassroomPresenter to enable users to create presentations on the XO and to record audio descriptions of the slides.
GNU General Public License v2.0
0 stars 6 forks source link

flake8 fixes #18

Open Saumya-Mishra9129 opened 4 years ago

Saumya-Mishra9129 commented 4 years ago

@quozl @pro-panda Please Review

quozl commented 4 years ago

Interesting, thanks. I can see what you did. I can see what triggered your change; the flake8 F821. But I can't see how you came to the conclusions as to what was needed to change or if the change could be tested. I'm not familiar with the activity or how to test it.

I see again that this repository has truncated commit history. http://activities.sugarlabs.org/en-US/sugar/addons/versions/4196 has the old versions listed in case further analysis is helpful..