tod
/
odin-default-js-exercises
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity
278 Commits 18 Branches 0 Tags 7.2 MiB
Commit Graph

278 Commits

Author SHA1 Message Date
plskz 8224cacfd4 Fix broken link 2021-04-04 22:49:14 +08:00
dependabot[bot] b0b57e6d56
Bump y18n from 3.2.1 to 3.2.2 in /generator-exercise 
Bumps [y18n](https://github.com/yargs/y18n) from 3.2.1 to 3.2.2.
- [Release notes](https://github.com/yargs/y18n/releases)
- [Changelog](https://github.com/yargs/y18n/blob/master/CHANGELOG.md)
- [Commits](https://github.com/yargs/y18n/commits)

Signed-off-by: dependabot[bot] <support@github.com>
 2021-03-29 16:32:02 +00:00
Michael Frank 2eb02ea212 update tests, multiple exercises 2021-03-10 00:12:25 +13:00
Michael Frank b9983073cc ignoring node_modules 2021-03-04 11:07:18 +13:00
Michael Frank e73c68fc01 Update README with Jest specific language. Update some spec files with new syntax 2021-03-04 10:42:39 +13:00
Michael Frank 91d50288b2 Change references from Jasmine to Jest in main readme 2021-03-03 23:19:51 +13:00
Michael Frank 7a001e0fdb run jest-codemods on .spec.js files, move generator-exercises back in 2021-03-03 15:25:29 +13:00
Michael Frank ecd82a4d52 Removed generator-exercises folder as it breaks jest-codemods 2021-03-03 15:22:27 +13:00
Michael Frank b8b23686e3 Added jest 2021-03-03 15:13:24 +13:00
Carlos Diaz 42076e7424
Merge pull request #124 from TheOdinProject/twalton83-patch-1 
update cloning link
 2021-01-18 12:34:35 -08:00
Tatiana 4c771f2e05
update cloning link 2021-01-18 12:31:31 -08:00
Tatiana 452caf2783
Merge pull request #123 from GroundedCastle/master 
Updated README
 2021-01-15 08:20:40 -08:00
GroundedCastle 35781bd985
Update README.md 2021-01-12 15:17:56 -06:00
GroundedCastle 6e0ae18d37
Updated README 
Made wording more generic (removed wrongly stated `cd exerciseName`), updated "The first exercise..." text in line 17.
 2021-01-12 15:17:20 -06:00
GroundedCastle 8992dbff97
Removed double spaces 2021-01-12 15:03:42 -06:00
GroundedCastle 13e75df196
Updated "To" as a test for making changes 2021-01-12 15:00:50 -06:00
GroundedCastle e240670a8e
Update README.md 2021-01-11 18:43:56 -06:00
GroundedCastle f06258377f
Merge pull request #1 from GroundedCastle/GroundedCastle-patch-1 
Updated Cogency, Syntax, and Clone Explanation
 2021-01-11 18:28:13 -06:00
GroundedCastle d6977b1f98
Updated Cogency, Syntax, and Clone Explanation 
The description said to install "NPM/Node." In the link that's provided, the author says that Node is being installed and specifically states that NPM is not being installed. It's more accurate to just say Node is being installed for now to avoid permission issues. Added colon for a list. Removed first person to create a consistent voice. Explained what cloning was because that's not covered up to this point in Foundations.  Made the capitalization consistent with the rest of the course.
 2021-01-11 17:23:34 -06:00
Tatiana 9a40c04726
Merge pull request #121 from vrarora/patch-3 
Update Leap Year Definition and grammar
 2021-01-10 05:27:41 -08:00
vrarora 6793f8722a
Update Leap Year Definition and grammar 
I got confused after seeing the code in solution that definition and solution didn't match so this is what I found on Wikipedia "(except for years evenly divisible by 100, which are not leap years unless evenly divisible by 400)" but earlier it was written here "except for years which are both divisible by 100 and not divisible by 400." 

I maybe wrong here, not knowing that these two statements maybe are indicating two different things, if that's the case, then feel free to not apply the changes and do let me know. 😄 

P.S. - I corrected "effects" to "affects" too.
 2020-12-23 17:06:25 +05:30
xandora d5a2b0ed0a
Merge pull request #120 from vrarora/patch-1 
Update helloWorld.js
 2020-12-23 07:27:26 +13:00
vrarora 19f6888688
Update helloWorld.js 
I added the answer instead of empty strings as solution must be added in the solutions branch.
 2020-12-22 14:01:32 +05:30
Briggs Elsperger 7e04012d05
Update README.md 2020-12-09 14:29:25 -08:00
Cody Loyd e5a4a1d2eb
Update caesar.js 
lol i'm dum
 2020-11-16 12:25:23 -06:00
Cody Loyd bde09dacbd
Update caesar.js 2020-11-16 12:24:26 -06:00
Marvin Gay 2c5bb33cfc
Update node installation link 2020-10-28 22:00:21 -04:00
Marvin Gay 5f32e12451
Merge pull request #116 from Abhineet0011/master 2020-10-18 11:26:53 -04:00
Marvin Gay 2563ec716b
Additional revisions to README 2020-10-17 19:43:29 -04:00
Abhineet0011 1c4b1a667e
Update readme.md 2020-10-17 21:20:36 +05:30
Briggs Elsperger cef55bcfb7
Merge pull request #115 from GraceSchwar/master 
Added hint to README.md Instructions
 2020-10-16 11:44:43 -06:00
GraceSchwar 7d37e6d67a
Update README.md 2020-10-16 13:44:16 -04:00
GraceSchwar 074b3b7481
Update README.md 2020-10-16 13:26:26 -04:00
Briggs Elsperger effa0cd70a
Merge pull request #114 from TheOdinProject/I3uckwheat-patch-2 
Update repeatString.spec.js
 2020-10-14 15:18:32 -06:00
Briggs Elsperger 6e02f72a72
Update repeatString.spec.js 2020-10-14 16:14:33 -05:00
Briggs Elsperger 8eeb98ee71
Merge pull request #113 from mangonaise/patch-1 
sumAll solution: check if min/max are integers, not just numbers
 2020-10-12 10:38:12 -05:00
liam robertson 2457cb4099
sumAll solution: check if min/max are integers, not just numbers 2020-10-10 19:06:51 +01:00
Cody Loyd 704a0f1826
Merge pull request #81 from Musabii/patch-1 
added a test and changed an old one
 2020-09-14 10:20:16 -05:00
Cody Loyd 314a795e52
Merge pull request #82 from Musabii/patch-2 
changed the order of if conditions
 2020-09-14 10:19:40 -05:00
Marvin Gay 4e004bdb1f
Merge pull request #76 from kyriri/patch-1 
Further clarify guidelines
 2020-09-13 21:14:20 -04:00
Musabii f9a972ec16
changed the order of if conditions 
it flows better logically to first check if the inputs are a number than to check if they are negative.
 2020-06-25 11:58:02 +02:00
Musabii 4d82ea16db
added a test and changed an old one 
The second test would originally test if it works with upper-case letters and punctuation. but the description of the test is "works with punctuation". I think it is better to either put both in the description or make a new test that solely tests each different thing.
 2020-06-25 11:17:48 +02:00
kyriri da45b701d4
Further clarify guidelines 
Everybody in the Southern hemisphere experiences winter in July and that's normal, so the original remark added an unnecessary (and off-topic) layer of confusion.
 2020-05-13 20:17:22 -03:00
Briggs Elsperger 888383a44e
Merge pull request #74 from imrogerjiang/new-tests 2020-05-09 12:16:42 -05:00
Roger 5817ea2140 Added blank string test to repeatString 2020-05-08 16:59:37 +10:00
Roger 3758497cb2 Added blank string test to reverseString 2020-05-08 16:56:55 +10:00
Roger 68691ab7a6 Added same type test to removeFromArray 2020-05-08 16:54:31 +10:00
Briggs Elsperger c0d736a524
Merge pull request #73 from Timework/patch-1 2020-05-02 21:29:46 -05:00
Christopher Martinez fb5677ee8f
Changed let to const and added some comments. 
I changed the let to a const and added some explanation to how the test is conducted in comments included in the test.
 2020-05-02 09:04:40 -05:00
Christopher Martinez 312103dc4f
Added random test 
I added a random test getting a number from 1 to 1000. You said you wanted comments on it but I was not sure where to put them. I added the global modifier to the recommended regex in order for it count all the words properly.
 2020-05-01 12:44:55 -05:00