JavaScript

algorithm/JavaScript

Fork 0

mirror of https://github.com/TheAlgorithms/JavaScript.git synced 2026-03-13 15:21:15 +08:00

Commit Graph

Author	SHA1	Message	Date
arthurvergacas	fa1abf4e1c	Minor fixes - Add some explanation and links about Carmichael Numbers - Remove explanation about in-built function Math.random()	2021-10-16 11:37:15 -03:00
arthurvergacas	ffb138adb1	Minor fixes - Add default number of iterations - Correct "corner cases" to "edge cases" - Make clear which bounds are inclusive and which are exclusive	2021-10-15 12:21:48 -03:00
Roland Hummel	c5e44d498a	A few suggestions / modifications / fixes It seems you've accidentally swapped the implementation and the test file :) The overall comment describing the algorithm (VERY nice doc, by the way) is not "proper" JSdoc => only one leading asterisk. It's generally considered good style to start a comment block (both JSdoc and regular comments) with a single, short sentence. Further down, there were some git hiccups, most likely caused by merge conflicts?	2021-10-14 22:32:56 +02:00

Author

SHA1

Message

Date

arthurvergacas

fa1abf4e1c

Minor fixes

- Add some explanation and links about Carmichael Numbers
- Remove explanation about in-built function Math.random()

2021-10-16 11:37:15 -03:00

arthurvergacas

ffb138adb1

Minor fixes

- Add default number of iterations
- Correct "corner cases" to "edge cases"
- Make clear which bounds are inclusive and which are exclusive

2021-10-15 12:21:48 -03:00

Roland Hummel

c5e44d498a

A few suggestions / modifications / fixes

It seems you've accidentally swapped the implementation and the test file :)

The overall comment describing the algorithm (VERY nice doc, by the way) is not "proper" JSdoc => only one leading asterisk. It's generally considered good style to start a comment block (both JSdoc and regular comments) with a single, short sentence.

Further down, there were some git hiccups, most likely caused by merge conflicts?

2021-10-14 22:32:56 +02:00

3 Commits