-
Notifications
You must be signed in to change notification settings - Fork 18
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
Round up temperature? #101
Comments
I think round-up looks better than decimals as (at least in the U.S.) the convention is to round w weather temperatures to the largest whole number. I can implement this if we want to move forward @digilou. |
That'd be great @M0nica! I think it's a nice touch from my perspective. |
I tried assigning this issue to you @M0nica , but somehow couldn't find your name in the assignees. Did you get an invite to join the team and did you accept it? |
@M0nica <https://github.com/M0nica> please look for an invite and accept :)
…On Fri, Mar 30, 2018 at 12:30 AM, Khusbu Chandra ***@***.***> wrote:
I tried assigning this issue to you @M0nica <https://github.com/M0nica> ,
but somehow couldn't find your name in the assignees. Did you get an invite
to join the team and did you accept it?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#101 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AIGoeo22PBF78HgDPxoRnu5WR-1LCljLks5tjbTbgaJpZM4S6ecH>
.
|
Thanks @tanyagupta @khusbuchandra @digilou! I accepted the invite and self-assigned this issue. I am traveling and probably won't be able to work on this until next week but would love to contribute. |
@M0nica can you still work on this? |
Yes @digilou I can work on this! |
Should we round off the current temperature?
The text was updated successfully, but these errors were encountered: