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
197a716eb0
reword test description to be less tautological and more clear
2023-04-06 18:41:30 -06: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
e416717ba9
Remove test skips for solutions
2023-02-01 18:58:58 -05:00
2ec0f4344d
Updated file paths
2023-02-01 18:53:54 -05:00
84d537f130
Correct minor spelling error
2023-01-29 18:10:11 -04:00
83d3dcd1dd
Add test for strings with numbers
...
Proposed by @thatblindgeye.
2023-01-28 16:26:19 +02:00
a092bf0559
Merge branch 'main' into update_solutions
2023-01-21 12:55:06 -05:00
4a112362c8
Update files
2023-01-21 12:53:41 -05:00
8746ce056a
Merge pull request #299 from Asartea/asartea-patch-1
...
07_tempConversion: Update tempConversion function's naming
2022-11-19 10:39:52 -05:00
9214f20afd
restore initial test state
2022-11-12 21:07:47 +01:00
0b218347ff
Missed README.md
2022-11-12 20:59:52 +01:00
374252c302
Update tempConversion function's naming
2022-11-12 20:49:28 +01:00
a05d4d60c8
PR Template: Flip order of checklist and details ( #289 )
...
* PR Template: Flip order of checklist and details
Because:
If the details are first, they are viewable in Discord's webhook.
This PR:
* Moves the detail sections to the beginning of the template
* Moves the checklist section to the end of the template
* Adds an issue section to link the issue
* Adds the Because, This PR, and Issue sections to checklist
* Reduce wording that is not essential
2022-10-21 21:43:16 -05:00
0747078d97
Updated Installing Node JS lesson URL
...
Because:
* We have updated our lesson urls
2022-03-27 13:51:06 +01:00
2586a8460c
Merge pull request #231 from TheOdinProject/thatblindgeye-patch-2
...
Templates: Create PR and Issue files
2022-02-22 17:01:28 -05:00
fb1a2db8d7
Add solution directories for exercises
2022-02-20 14:07:44 -05:00
649f0e3771
Update PULL_REQUEST_TEMPLATE.md
2022-02-20 13:25:26 -05:00
f82c72f760
Update bug_report.md
2022-02-20 13:21:49 -05:00
f002e3d6d0
Update feature_request.md
2022-02-20 13:20:37 -05:00
1971712fde
Merge pull request #230 from TheOdinProject/thatblindgeye-patch-1
...
README: Make more consistent with other repos
2022-02-17 07:22:10 -05:00
67d4d0afec
Update README.md
2022-02-16 22:01:13 -05:00
076059551a
Update README.md
2022-02-16 17:41:44 -05:00
2b54b1d682
Merge branch 'main' into thatblindgeye-patch-1
2022-02-16 17:26:00 -05:00
2ebbb5a925
Merge pull request #232 from jernestmyers/fix/update-readme
...
Remove irrelevant text from README about order of completion
2022-02-16 08:55:14 +05:00
edb9c244b5
Remove irrelevant text about order of completion
2022-02-14 16:05:00 -05:00
417ee0d6e5
Create feature_request.md
2022-02-14 15:09:44 -05:00
baa5b0a761
Create bug_report.md
2022-02-14 15:09:20 -05:00
6dd594c900
Create PULL_REQUEST_TEMPLATE.md
2022-02-14 15:07:50 -05:00
1b43007a26
Update README.md
2022-02-14 14:59:03 -05:00
2f9f431194
Merge pull request #227 from Mclilzee/main
2022-02-13 09:22:26 +13:00
1acdfccb5d
Fix palindromes false positive test
2022-02-08 20:52:06 +01:00
82db5eac8b
Merge pull request #215 from programmurr/.../...new-node-install
2022-01-28 09:45:37 +13:00
efdfbd441c
Merge pull request #206 from chagstep1/patch-1
2022-01-27 14:32:10 +13:00
b1a0bf97c3
Merge pull request #193 from HiddenOgre28/main
2022-01-27 14:30:10 +13:00
MarLatte
Eric Olkowski
thatblindgeye
Ryan McEntire
Kevin Mulhern
Alex-Nemet
Asartea
jhylacey
01zulfi
Telcean Marius-Andrei
Rachel Moser
Jeremy Myers
Michael Frank
Mclilzee