-
Notifications
You must be signed in to change notification settings - Fork 13
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
Fix to the warning, using string sum instead #328
Fix to the warning, using string sum instead #328
Conversation
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.
A few changes needed:
- add changelog entry
- keep the execution output
- update the zip if you updated anything in the notebook
Also it would be help that you can pin-point which line did you change?
6c3b2a9
to
7157a99
Compare
8c144d6
to
b401b43
Compare
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.
Checked the output against the previous and everything looks good. If you can address the issues @longshuicy identified, then I think it'll be ready to merge. |
@navarroc @longshuicy I think it is ready now to be merged :) |
Can you remove the .ipynb_checkpoints folder from each zip file? |
@navarroc removed 👍 |
Sorry to nitpick, but can you remove the hidden .DS_Store file in the Galveston zip file? Thank you! |
Can you remove the __MACOSX folder in the zips? |
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.
zipfiles look good now
No description provided.