305ad2dfc8
Update 06_leapYears README.md with a reference
...
I updated the README with an external link outlining the nested rules for leap years. The present problem statement, borrowed from Chris Pine, does not clearly state these rules, which is indeed confusing.
To scaffold the learner's journey, it would be useful to add a reference that walks them through the leap year rules with examples. It difficult to stick to the practice exercise if the learner has to shoot in the dark without knowing the exact rules of the game.
2024-11-16 22:26:40 +05:30
203fd76a54
Merge pull request #501 from o1Suleyman/patch-2
...
Whitespace and capitalization correction
2024-10-30 16:18:23 +00:00
406b4c411e
Whitespace and capitalization correction
2024-10-26 13:22:50 +04:00
e51671ec4f
Calculator Factorial: Remove redundant guard clause ( #499 )
2024-10-18 19:40:42 +01:00
a12f3116fd
08_calculator:Add more test cases for subtract and power ( #494 )
2024-09-23 18:09:04 +01:00
7ff69d817d
Refactor issue markdowm to YAML templates ( #491 )
2024-09-04 21:16:13 +01:00
29dd5a4a2e
feat(repeatString): enforce manual implementation via loops ( #460 )
...
Intent is to practise using loops rather than wrap the built-in `repeat` String method.
2024-08-10 18:00:39 +01:00
0c5480d00e
sumAll: Unify tests between solution and non-solution spec file ( #488 )
2024-07-17 14:02:56 +01:00
3c636bebee
Leap Years: Use more appropriate solution ( #483 )
...
Replacement solution is closer to what most learners might think of due to the instructions' hints, and concepts they'll have been reasonably exposed to so far in the curriculum.
The original solution has been a curveball for quite a few people.
Co-authored-by: Josh Veneziano <jveneziano@proton.me >
2024-06-27 12:25:01 +01:00
435b88bf19
Merge pull request #481 from TheOdinProject/dependabot/npm_and_yarn/braces-3.0.3
...
Bump braces from 3.0.2 to 3.0.3
2024-06-11 15:16:57 -04:00
6e3d7d2d19
Bump braces from 3.0.2 to 3.0.3
...
Bumps [braces](https://github.com/micromatch/braces ) from 3.0.2 to 3.0.3.
- [Changelog](https://github.com/micromatch/braces/blob/master/CHANGELOG.md )
- [Commits](https://github.com/micromatch/braces/compare/3.0.2...3.0.3 )
---
updated-dependencies:
- dependency-name: braces
dependency-type: indirect
...
Signed-off-by: dependabot[bot] <support@github.com >
2024-06-11 18:43:45 +00:00
82712dbac1
Merge pull request #478 from pixie-cheeks/check_node_ver
...
Project Root: Add minimum npm & node version requirements
2024-06-11 14:43:09 -04:00
c79a878fce
Add minimum npm & node version requirements
...
This change will prevent any old node or npm version from installing any
packages. With this, npm will throw an error and exit immediately.
This is to prevent the accidental use of unsupported versions which
might cause some issues that are hard to pin-point.
2024-06-04 15:43:52 +05:30
2154dcec4a
Changed number to integer in sumAll ( #477 )
2024-06-01 12:43:35 +01:00
312bd8a1d9
05_sumAll: Update README.md ( #475 )
...
* Change 'integers' to 'positive integers' in question statement
* requested changes
* requested changes
* requested changes
2024-05-25 13:36:54 +01:00
d67e7a3ecb
03_reverseString - update punctuation test case to include comma ( #467 )
2024-05-06 19:40:50 +01:00
097a73634f
Merge pull request #465 from JoshDevHub/change_style_of_notes
...
Change styling of notes for avoiding PRs and error messages
2024-05-04 14:12:04 -06:00
6f9080c3db
Change styling of notes for avoiding PRs and error messages
2024-05-04 13:06:04 -07:00
1695abe9d4
Exercise 12: Provide alternative solution ( #459 )
...
* feat(12): improve solution by making it more cleaner
* refactor(12): rename `array` to `people`
* feat(12): improve explanation of `??=`
Co-authored-by: MaoShizhong <122839503+MaoShizhong@users.noreply.github.com >
* feat(12): include both solutions
* feat(12): hint to use various array methods
* refactor(12): add semicolon
Co-authored-by: MaoShizhong <122839503+MaoShizhong@users.noreply.github.com >
* feat(12): use block comments instead of line for readability
Co-authored-by: MaoShizhong <122839503+MaoShizhong@users.noreply.github.com >
* refactor(12): remove unnecessary comment
Co-authored-by: MaoShizhong <122839503+MaoShizhong@users.noreply.github.com >
---------
Co-authored-by: MaoShizhong <122839503+MaoShizhong@users.noreply.github.com >
2024-05-03 16:30:56 +01:00
45a5ced879
Improve README verbiage ( #453 )
...
This is sort of a grab-bag of things that I found while reading the README.
Each change is fairly small so it seemed best to combine them under one commit.
* Add link to The Odin Project home page in 1st paragraph.
* Remove a few contractions. (Contractions are not used consitently here,
so it was easier/better to just remove the few that popped up.)
* Fix some awkward phrasing.
2024-05-03 02:16:06 +01:00
930673d9c8
feat(08): remove recursive solution ( #457 )
...
Not appropriate to expose learners to at this point in the curriculum
2024-05-03 02:13:38 +01:00
1afbcf8644
feat(04): use const instead of var in solution ( #456 )
2024-05-03 02:12:28 +01:00
1c322ddb47
Merge pull request #420 from kumang-subba/repeatstring_test_alternate_strings
...
added alternate repeatString test strings
2024-04-20 08:14:28 -04:00
9a5cdecf5b
Update 02_repeatString/solution/repeatString-solution.spec.js
...
Co-authored-by: Nikita Revenco <154856872+nikitarevenco@users.noreply.github.com >
2024-03-24 07:09:52 +02:00
8209109149
Merge pull request #436 from TheOdinProject/plopGenerator
...
Replace generator-exercise with plop generator
2024-03-03 13:33:51 -05:00
ffe7f1484a
Add newline at end of generated file content
2024-03-03 10:50:22 -05:00
7faf5806f0
Update readme, add case-anything dep
2024-03-03 10:46:06 -05:00
d063f5403e
Replace generator-exercise with plop generator
2024-03-03 10:46:06 -05:00
461c852f98
Use const for palindrome solution variable ( #439 )
2024-03-03 01:44:34 +00:00
f215901131
Replace palindromes exercise solution with non-regex version ( #438 )
...
Adapted the non-regex solution provided in the solutions branch (which
was not brought over to the main branch's solution file) to pass the
current test suite.
Cleaned formatting of comments and reworded for clarity.
2024-03-02 13:57:16 +00:00
a7bcd235d8
Merge pull request #435 from manny53365/07_tempConversion-update-readme-for-clarity
...
07_tempConversion updated the readme file for clarity on the assignment
2024-02-13 17:25:31 -05:00
b09216355f
Merge pull request #428 from donRehan/main
...
12_findTheOldest fix ambiguity and remove redundant test description
2024-02-13 17:24:09 -05:00
3006bca955
updated the readme file for clarity on the assignment
2024-02-14 03:43:16 +05:30
8bc6e47821
Update findTheOldest test to clear ambiguity
2024-02-12 14:36:33 +02:00
7a097ac960
Merge branch 'main' into repeatstring_test_alternate_strings
2024-02-10 18:44:14 +02:00
a84f55bd60
Merge pull request #433 from manny53365/fix-typo-in-test-case
...
corrected typo in the test case for 02_repeatString
2024-02-09 08:53:24 -05:00
656d184963
Merge branch 'main' into repeatstring_test_alternate_strings
2024-02-09 13:57:41 +02:00
7512dd60bd
corrected typo in the test case
2024-02-08 22:20:22 +05:30
d740f2de09
Merge pull request #432 from damon314159/patch-1
...
04_removeFromArrray: missing test case
2024-02-05 15:14:54 +00:00
3a49a90dd7
feat: extra test case
2024-02-05 10:10:00 +00:00
eac90c5621
Merge pull request #404 from TheOdinProject/dependabot/npm_and_yarn/babel/traverse-7.23.2
...
Bump @babel/traverse from 7.22.11 to 7.23.2
2024-02-03 15:14:01 -05:00
f3a6283a93
12_findTheOldest fix ambiguity and remove redundant test description
...
fix ambiguity in test description , old was 'finds the person with the
greatest age if someone is still living.'
remove redundant test that checks for 'finds the person with the
greatest age if the OLDEST is still living' since both this and the
above test for ability to check if there is no death year field.
2024-01-23 21:16:03 +02:00
77363aa547
Merge pull request #424 from MaoShizhong/chore/improve-readme-wording-on-provided-solution
...
Repo README: Improve wording regarding TOP provided solution
2024-01-19 12:42:15 -05:00
c87cd1c9d8
Use lazy numbering as per style guide
2024-01-19 13:28:31 +00:00
d3a748812b
Update wording
2024-01-19 13:24:18 +00:00
311ae2fc33
Merge pull request #342 from TheOdinProject/bycdiaz-patch-2-1
...
Adds hint to exercise
2024-01-19 08:14:51 -05:00
f682844f77
Update 04_removeFromArray/README.md
2024-01-19 08:13:57 -05:00
5f7e20f149
Merge pull request #426 from kennotfindsymbol/patch-1
...
02_repeatString/solution/repeatString-solution.spec.js: Fix typo
2024-01-19 07:28:15 -05:00
e7f5d91bd1
Fix typo
...
I think it should be 'randomly' instead of 'randomaly' on line31.
2024-01-19 19:19:49 +08:00
f4293596b7
Improve wording in README regarding TOP provided solution
...
Same change made in parallel with the css-exercises repo to unify
wording.
2024-01-15 09:06:44 +00:00
Bhagyesh Pathak