Labels Milestones

finalize-db-ef-player #110

Merged

alexis.drai merged 15 commits from finalize-db-ef-player into main 3 years ago

Conversation 0 Commits 15 Files Changed 12

alexis.drai commented 3 years ago

Owner

There is no content yet.

alexis.drai added 15 commits 3 years ago

6c03fc4586 ⚗️ Try things out

90c1eba43b ✏️ Fix typo

722a4314fa ⚗️ Try things out, in accordance with wiki UML

cfb495cd68 argh

901a1128e9 🩹 Try-catch db connectionsin prototype

621d6c3bc2 ✨ ⚗️ Add features to prototype, test DB further

2fae748af9 👔 Implement PlayerDbManager

912f6f6627 ✅ Add tests for GetOneById()

4632516d2c 🧐 Switch CLRF thing ...

I really should look into that

c100c20a39 🔥 Improve some model UT

1604fece2f :refactor: Use Guid.NewGuid() instead of hardcoding them ...

obviously ^^'

9a453da7bc 🔧 Disable nullable reference types in the Data project

d8fead1051 ✅ Fully test PlayerDbManager, and improve it

5b4c2469c9 ✅ Test the DbStub a little bit too

85a1b67dd6 📈 Shrink untested lines to make those metrics shine

alexis.drai merged commit 0a9ab6a39a into main 3 years ago

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

Merge pull request 'finalize-db-ef-player' (#110) from finalize-db-ef-player into main

continuous-integration/drone/push Build is passing

Details

The pull request has been merged as 0a9ab6a39a.

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#110

Write Preview

Loading…

Cancel

Save

There is no content yet.