London|25-ITP-September|Alexandru Pocovnicu|Sprint 2 #814

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Open

alexandru-pocovnicu wants to merge 21 commits into CodeYourFuture:main from alexandru-pocovnicu:Sprint-2

alexandru-pocovnicu commented Nov 3, 2025 •

edited

Loading

Learners, PR Template

Self checklist

I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title
My changes meet the requirements of the task
I have tested my changes
My changes follow the style guide

Changelist

implemented functions using TDD

alexandru-pocovnicu added 21 commits

October 24, 2025 10:12


          Add initial test file for mean calculations

71fa218


          Add mean.js file for mean calculations

927769b


          Remove mean.js and mean.test.js files

a393af9


          Fix address object access to correctly log house number

256a0bb


          Fix loop iteration to correctly log author object properties

93dca5e


          Fix recipe logging to display ingredients on separate lines

538d536


          Fix recipe logging format to display title, serves, and ingredients i…

608a2f2

…n a single line


          Add test case for contains function with empty object

3a82eed


          Implement contains function to check for property existence in an object

dbd5551


          Add additional test cases for contains function to cover non-existent…

1769faf

… properties and invalid parameters


          Add error handling for non-object parameters in contains function

8fd279f


          Refactor test cases for contains function for improved readability an…

1ff152e

…d consistency


          Fix formatting inconsistencies in contains function for improved read…

2982bf2

…ability


          Implement test for createLookup function to validate country-currency…

cb3b23c

… code mapping


          Implement createLookup function to map country to currency

e56a2bf


          Add tests for tally function to validate behavior with empty and inva…

3f4a115

…lid inputs


          Implement tally function to count occurrences of elements in an array…

493e3f1

… with input validation


          Refactor query string tests for improved clarity and structure

577ae89


          Refactor parseQueryString function for improved handling of empty pai…

99caad7

…rs and key-value parsing


          Fix invert function to correctly swap keys and values in the object

f998538


          Add test for invert function to verify key-value swapping behavior

e098bba

github-actions bot commented Nov 3, 2025

Your PR's title isn't in the expected format.

Please check the expected title format, and update yours to match.

Reason: Wrong number of parts separated by |s

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

alexandru-pocovnicu added the Needs Review label

github-actions bot commented Nov 3, 2025

Your PR's title isn't in the expected format.

Please check the expected title format, and update yours to match.

Reason: Wrong number of parts separated by |s

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

4 similar comments

github-actions bot commented Nov 3, 2025

Your PR's title isn't in the expected format.

Please check the expected title format, and update yours to match.

Reason: Wrong number of parts separated by |s

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

github-actions bot commented Nov 3, 2025

Your PR's title isn't in the expected format.

Please check the expected title format, and update yours to match.

Reason: Wrong number of parts separated by |s

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

github-actions bot commented Nov 3, 2025

Your PR's title isn't in the expected format.

Please check the expected title format, and update yours to match.

Reason: Wrong number of parts separated by |s

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

github-actions bot commented Nov 3, 2025

Your PR's title isn't in the expected format.

Please check the expected title format, and update yours to match.

Reason: Wrong number of parts separated by |s

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels