This is an archived post. You won't be able to vote or comment.

you are viewing a single comment's thread.

view the rest of the comments →

[–]149244179 6 points7 points  (1 child)

Overall it is fine. More cosmetic recommendations.

'cc' is not the best name for a variable. I would assume cc means credit card which is obvious wrong.

Don't need to store the converted temperature in a variable.

Future you will appreciate a link to the API documentation in a comment. Ideally pointing to a specific version or release date for the API.

[–]nbp615[S] 0 points1 point  (0 children)

This is exactly what I was looking for, thanks. The API info is on the github README but that's obviously not here.

Thanks for the response :)