generated from CodeYourFuture/Module-Template
-
-
Notifications
You must be signed in to change notification settings - Fork 240
Manchester |25-ITP-Sep|Mahtem T. Mengstu |Sprint 3|coursework/sprint-3-implement-and-rewrite #851
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.
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
Open
Mahtem
wants to merge
19
commits into
CodeYourFuture:main
Choose a base branch
from
Mahtem:coursework/sprint-3-implement-and-rewrite
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
19 commits
Select commit
Hold shift + click to select a range
f723809
Sprint-3-implement 1-get-angle-type.js all assertions and necessary f…
Mahtem ac08757
// Sprint-3-implement 1-is-proper-fraction.js all assertions and nece…
Mahtem 2c28a87
Corrected an error in the code
Mahtem 5d979ed
Modified -implement -is-proper-fraction.js
Mahtem 5f2e946
Sprint-3 -3-get-card-value.js Functions, assertion and invalid rank t…
Mahtem ba5537d
Sprint-3 rewrite-tests-with jest 1-get-angle-type-test.js tests condu…
Mahtem 26e660e
Modified Typo
Mahtem 4930742
// Sprint-3 ewrite-tests-with-jest/2-is-proper-fraction.test.js. rewr…
Mahtem c68a37c
Sprint-3 rewrite-tests-with-jest/2-get-card-value.test.js. rewrote te…
Mahtem b57c404
Few modifications done.
Mahtem 0a1ed8c
Little change made
Mahtem 9abc5a9
2-is-proper-fraction.js Updated the function and has been able to ha…
Mahtem 608a309
Functions, assertions, and invalid ranks tested. Modified the if stat…
Mahtem b0eefec
Testing modified to cover broader scenarios that cover all possible c…
Mahtem 6041f59
Corrected a misleading comment
Mahtem e84da29
Completed the incomplete description and added a test for multiple va…
Mahtem f8bbdf0
Removed misleading description
Mahtem 2934b0e
Added test coverage cases
Mahtem 74b4cc1
Modified
Mahtem File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could remove code that's no longer needed (to keep the code clean).
The function is not expected to validate the suit character. On line 5, mentioning the suit character in the test description could mislead the person implementing the function into thinking the function needs also to check the suit character.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good, that's right, I have removed misleading description. and changed it.