Labels Milestones

🐛♻️✅ Fix-uts #51

Merged

alexis.drai merged 6 commits from fix-uts into main 3 years ago

Conversation 0 Commits 6 Files Changed 4

alexis.drai commented 3 years ago

Owner

There is no content yet.

alexis.drai added 6 commits 3 years ago

dcb9de97e5 🩹 Pass params by vals instead of by refs ...

if we pass parameters by reference, it must be because we want to
change them in the method and to keep those changes.
This seems like a different situation, so we should be able
to pass the objects by value.

f1404f829d 🐛 Add validation and delegation to Player.Remove()

4472c56354 ✅ Improve Update() test after bugfix

085e27122a ✏️ Fix typo

e3a86a67b9 ✅ Generalize from HashSets to IEnumerables or Collections

10690dc991 ♻️ Refactor, D.R.Y., use "expected" and "actual"

alexis.drai merged commit fd661d5b5c into main 3 years ago

alexis.drai deleted branch fix-uts 3 years ago

alexis.drai referenced this issue from a commit 3 years ago

Merge pull request '🐛♻️✅ Fix-uts' (#51) from fix-uts into main

continuous-integration/drone/push Build is passing

Details

The pull request has been merged as fd661d5b5c.

Sign in to join this conversation.

Reviewers

Request review

No reviewers

Labels

Apply labels

Clear labels

bug
Something is not working code smell
Not exactly a bug, but this code will cause problems down the line doc
About documentation duplicate
This issue or pull request already exists fixed
This issue has been fixed, but should not be closed yet help wanted
Need some help invalid
Something is wrong low priority
This issue has a lower priority meta
About issues / tickets themselves nice to have
This would be nice to have, but not essential performance
This is to improve performance in some way prerequisite
These issues needs to be resolved before moving on to any other issue question
More information is needed required
New requirement setup
About the project tree or structure testing
About code quality and testing waiting for release
This issue will need to be resolved when we release, but not before wontfix
This won't be fixed

No Label bug code smell doc duplicate fixed help wanted invalid low priority meta nice to have performance prerequisite question required setup testing waiting for release wontfix

Milestone

Set milestone

Clear milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Assign users

Clear assignees

No Assignees

1 Participants

Notifications

Subscribe

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: alexis.drai/dice_app#51

Write Preview

Loading…

Cancel

Save

There is no content yet.