Skip to content

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

Merged
merged 8 commits into from
Oct 7, 2021

Conversation

SabariGanesh-K
Copy link
Contributor

I added a file which contains essential conversions needed to swap temperature units to one another.
Hope you like it :)

I added a file which contains essential conversions needed to swap temperature units to one another.
Hope you like it :)
Copy link
Member

@raklaptudirm raklaptudirm left a 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]>
@SabariGanesh-K
Copy link
Contributor Author

I will add tests file also.Kindly give me some time :) Will add it soon

@raklaptudirm raklaptudirm marked this pull request as draft October 5, 2021 07:12
@raklaptudirm
Copy link
Member

Please mark the Pull request as Ready for review once you have implemented the tests.

@raklaptudirm raklaptudirm added the in progress Being worked on label Oct 5, 2021
"test" folder was created and tests required for "TemperatureConversions" was added to it
@SabariGanesh-K
Copy link
Contributor Author

I have added the test file @raklaptudirm

@SabariGanesh-K SabariGanesh-K marked this pull request as ready for review October 5, 2021 13:04
@raklaptudirm
Copy link
Member

Please fix the tests, they are failing.

@raklaptudirm raklaptudirm added algorithm Adds or improves an algorithm feature Adds a new feature code style issue Failing style checks Reviewed changes required This pull request needs changes and removed code style issue Failing style checks in progress Being worked on labels Oct 6, 2021
@SabariGanesh-K
Copy link
Contributor Author

@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

SabariGanesh-K and others added 3 commits October 6, 2021 23:04
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
@raklaptudirm
Copy link
Member

The style checks are failing, please format your code again.

@raklaptudirm raklaptudirm added code style issue Failing style checks and removed changes required This pull request needs changes labels Oct 7, 2021
Hope this passes the test :)
@raklaptudirm raklaptudirm removed the code style issue Failing style checks label Oct 7, 2021
@raklaptudirm raklaptudirm merged commit 124d201 into TheAlgorithms:master Oct 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
algorithm Adds or improves an algorithm feature Adds a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants