* fix: consolidate/remove comments
* fix: remove => from comment
* fix: reverted changes to add same changes to another PR
* fix: removed 'the' from sentence
Co-Authored-By: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
* fix: removed 'the' from the sentence
Co-Authored-By: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
Co-authored-by: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
* fix: Chinese test suite
Add localeTiltes, descriptions, and adjust test text and testStrings to get the automated test suite working.
* fix: ran script, updated testStrings and solutions
* fix: improve Project Euler descriptions and test case
Improve formatting of Project Euler test descriptions. Also add poker hands array and new test case for problem 54
* feat: add typeof tests and gave functions proper names for first 100 challenges
* fix: continue fixing test descriptions and adding "before test" sections
* fix: address review comments
* fix: adjust grids in 18 and 67 and fix some text that reference files rather than the given arrays
* fix: implement bug fixes and improvements from review
* fix: remove console.log statements from seed and solution
Related to PR #38040
This PR is an attempt to consolidate/remove as many comments in the challenge seed code in the Responsive Web Design certification challenges, to be able to create a small translation lookup object for translating the English comments to other languages.
* Added meaning of ones in the matrix
* Apply suggestions from code review
Co-Authored-By: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
* fix: add slash to avoid markdown lint error
Co-authored-by: mrugesh <1884376+raisedadead@users.noreply.github.com>
Co-authored-by: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
* add: solution to project euler problem 20
* Update curriculum/challenges/english/08-coding-interview-prep/project-euler/problem-20-factorial-digit-sum.english.md
Accidentally added last line. Calling function is unnecessary in solution.
Co-Authored-By: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
* right place for starting braces
Co-Authored-By: Manish Giri <manish.giri.me@gmail.com>
* moved starting brace to previous line.
Co-Authored-By: Manish Giri <manish.giri.me@gmail.com>
Co-authored-by: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
Co-authored-by: Manish Giri <manish.giri.me@gmail.com>
* fix: added test to validate new array is returned
* fix: changed text to reflect changed test
Co-Authored-By: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
Co-authored-by: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
* fix: localise Heatmap to user's timezone
Rather than using ISO formatted date strings, this uses Date objects
for simplicity and to ensure that the heatmap is correct for the
timezone it is viewed in. It should also match the timeline which is
also localised to the viewing computer's timezone.
* test: update snapshot
react-calendar-heatmap's output depends on the timezone, which means
that snapshots can fail if the timezone changes. This sets the timezone
as UTC during client tests to avoid that problem.