bb78d163fc
Add note about packages out of date
...
Related to: NPM Vulnerabilities Warning #250
2023-08-24 21:35:07 -05:00
59d27bf89f
Update NPM packages to latest versions
...
Related to #250
2023-08-24 21:28:42 -05:00
9599e2ade3
Merge pull request #369 from cats256/cats256-branch
...
Reformat test parameters for consistency
2023-07-20 18:28:53 -04:00
44e39f0412
Update calculator-solution.js
2023-07-16 16:06:25 -05:00
1dae9df3ce
Update calculator-solution.spec.js
2023-07-16 16:02:28 -05:00
f855eb51b2
Update calculator-solution.spec.js
2023-07-16 16:02:03 -05:00
e10ee035ad
Update calculator.spec.js
2023-07-16 15:59:50 -05:00
e3810865cf
Merge pull request #378 from Roberra0/ex12_ReadmeClarified
...
Ex12 readme clarified
2023-07-12 08:27:24 +01:00
100f952f7a
Update README.md with minor word change
2023-07-10 17:21:52 -07:00
e6c4530aa9
Update ex12_findTheOldest README.md
...
Added clarification to instruct learner to look in test case to understand object structure
2023-07-10 17:09:29 -07:00
15f1b82b57
Merge pull request #370 from cats256/patch-2
...
Update fibonacci-solution.js
2023-07-07 08:47:24 +05:00
51572a070c
Update fibonacci-solution.js
2023-07-06 09:47:20 -05:00
f164d790da
Merge pull request #372 from Asartea/patch-1
...
Fix: change requirement to reflect solutions change
2023-07-06 09:11:44 +01:00
fcb1c4971a
Update fibonacci-solution.js
2023-07-05 09:15:36 -05:00
051c0ed9ca
CONTRIBUTING.md: repoint contributing links to .github/CONTRIBUTING.md ( #371 )
...
* repoint contributing links to .github/CONTRIBUTING.md
* drop mention of which repo it is
2023-07-05 22:41:21 +10:00
bab1364ea8
Fix: change requirement to reflect solutions change
2023-07-05 13:50:13 +02:00
76551b0e8a
Update fibonacci-solution.js
...
Better O(n) time
2023-07-04 12:07:04 -05:00
03e52ea9ee
Improve sum and multiply functions solution code
2023-07-04 11:55:08 -05:00
075fe8eea2
Reformat test parameters for consistency
2023-07-04 11:38:40 -05:00
5a7cd9b162
Merge pull request #367 from ManonLef/kbd
2023-07-02 09:01:22 +02:00
37f85db108
JS exercises README: change shortcuts to kbd format
2023-07-01 16:09:59 +02:00
61c86e11b6
Merge pull request #365 from Sama-004/patch-1
...
`03_reverString: Updated README.md to add unskipping the skipped test cases`
2023-06-28 14:22:37 +01:00
f98ee210e0
Made the file name sentence more readable
...
Made changes according to the review, made the file name more readable and some minor grammatical changes.
2023-06-28 17:34:00 +05:30
5c1853e1e9
Updated README.md to add unskipping the skipped test cases
...
Just like it was mentioned in the previous exercise to remove the .skip from test.skip() in the spec.js file.
2023-06-28 08:43:15 +05:30
25013df6ac
Merge pull request #357 from marlatte/fibonacci_palindromes_fixes
...
09_palindrome and 10_fibonacci: Update solutions
2023-06-08 07:15:50 -05:00
8692f0ea18
Merge pull request #364 from fruddenfeldt/dev
...
New solution to match updated test syntax
2023-06-07 14:56:36 -05:00
175ee761e1
Update calculator-solution.js
2023-06-07 19:58:54 +02:00
6b302e3783
Merge pull request #359 from fruddenfeldt/patch-1
...
Removed array syntax in multiply test
2023-06-07 08:45:00 -05:00
80f7881a26
New solution to match updated test syntax
...
See separate PR "Removed array syntax in multiply test #359"
https://github.com/TheOdinProject/javascript-exercises/pull/359
2023-06-06 23:01:36 +02:00
b6e9e2fac3
Removed array syntax in multiply test
...
The test for the 'multiply' function used array brackets [] for the input parameters, which caused the test to return an error when rest parameters (...args) are used in the function, like so:
const multiply = function(...args){
return args.reduce((acc, cur) => acc * cur);
}
2023-05-22 21:08:37 +02:00
fd1e1f93d1
Merge branch 'fibonacci_palindromes_fixes' of github.com:marlatte/javascript-exercises into fibonacci_palindromes_fixes
2023-05-20 03:10:37 -04:00
0d75cc0814
Update palindrome test to match solution test
2023-05-20 03:07:57 -04:00
e8fc8ce41e
Update palindromes to handle numbers better
...
The previous solution removed numbers entirely, whereas this one treats
them like letters and checks if they are evenly spaced.
More importantly, the old solution test seemed to check if the numbers
were palindromic, but because the solution replaced them with "", it
wasn't testing what it seemed to.
I added a new test to differentiate between the palindromic "rac3e3car"
and the non-palindromic "r3ace3car".
These changes also obviate the problems raised in Issue #355 .
2023-05-20 02:47:17 -04:00
41380593f7
Update palindromes to handle numbers better
...
The previous solution removed numbers entirely, whereas this one treats
them like letters and checks if they are evenly spaced.
More importantly, the old solution test seemed to check if the numbers
were palindromic, but because the solution replaced them with "", it
wasn't testing what it seemed to.
I added a new test to differentiate between the palindromic "rac3e3car"
and the non-palindromic "r3ace3car".
2023-05-20 02:37:41 -04:00
3256f980b0
Update Fibonacci solution, test to focus on arrays
...
Previous Fibonacci solution didn't use arrays, and since this is an
array-heavy section, it seemed better to have that be the method here.
Also, it accounts for entering 0 as an argument without having to add
any extra code, which takes care of some currently open issues.
Issues #192 and #236 are about the same thing.
I added a test for that as well.
2023-05-20 02:22:12 -04:00
3e530e3f61
Merge pull request #341 from thatblindgeye/updatePalindromes
...
09_palindromes: update solution test file
2023-04-08 15:51:19 -04:00
6bb8b1397d
Merge pull request #321 from Alex-Nemet/fix_typo_readme
...
Readme.md: fix spelling
2023-04-08 15:48:15 -04:00
045f1b0306
Update solutions file for palindrome
2023-04-08 15:44:15 -04:00
34d63cc59d
Merge pull request #320 from mariustelcean96/patch-2
...
09_palindromes: Add test for strings with numbers
2023-04-08 15:42:52 -04:00
029cf3ffcd
Merge pull request #338 from TheOdinProject/KevinMulhern-patch-2
...
Create LICENSE
2023-03-28 08:26:14 +01:00
378921289b
Create LICENSE
...
Because:
* This repo does not have a licence
This commit:
* Add the MIT licence
2023-03-25 15:49:30 +00:00
cface9db19
Merge pull request #337 from TheOdinProject/revert-license
...
Revert "Create LICENSE"
2023-03-25 15:48:02 +00:00
736cae8b06
Revert "Create LICENSE"
...
This reverts commit e63e0bf296
2023-03-25 15:46:09 +00:00
e63e0bf296
Create LICENSE
...
Because:
* This repo did not have a licence
This commit:
* Add MIT Licence
2023-03-25 15:44:27 +00:00
614b8b8c35
Merge branch 'main' into fix_typo_readme
2023-03-22 12:59:21 -07:00
75869d1912
Mark caesar as archived and move archived exercise to own folder ( #336 )
2023-03-18 16:45:37 -05:00
6f5f71ff5b
Remove redundant wording in reverse string README
2023-03-18 14:14:44 -05:00
ea1f23639c
Merge pull request #329 from TheOdinProject/test-workflow
...
Add circleci config file
2023-03-05 11:59:59 +00:00
99d0e8a71a
Add circleci config file
...
* We can use circleci to test all spec files in solution subdirectories
2023-02-20 19:10:10 +05:00
8dfb7e4508
Merge pull request #234 from thatblindgeye/update_solutions
...
Exercises: Add solution directories for all exercises
2023-02-20 08:42:44 -05:00
Brian Lister
Eric Olkowski
Will
Austin
Roberra Aklilu
01zulfi
Asartea
Manon
Manon
samanyu
Cody Loyd
fruddenfeldt
MarLatte
thatblindgeye
Kevin Mulhern
Alex-Nemet
jhylacey