-
-
Notifications
You must be signed in to change notification settings - Fork 5.7k
Added Temperature Conversions to Javascript #705
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
Conversation
I added a file which contains essential conversions needed to swap temperature units to one another. Hope you like it :)
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.
Please add tests in a separate jest
file. Look into the test
sub-directories for reference.
Suggestions saved Co-authored-by: Rak Laptudirm <[email protected]>
I will add tests file also.Kindly give me some time :) Will add it soon |
Please mark the Pull request as |
"test" folder was created and tests required for "TemperatureConversions" was added to it
I have added the test file @raklaptudirm |
Please fix the tests, they are failing. |
@raklaptudirm It was actually some parenthesis error. I fixed it. Actually I created test file with the help of already available file at "math" section. Should I import anything like mocha (for testing) in this folder which I created or is it configured ? Kindly check it and confirm |
Co-authored-by: Rak Laptudirm <[email protected]>
The code was modified to comply with rules and styes recommended by the repository
The code was modified to comply with rules and styes recommended by the repository
The style checks are failing, please format your code again. |
Hope this passes the test :)
I added a file which contains essential conversions needed to swap temperature units to one another.
Hope you like it :)