-
Notifications
You must be signed in to change notification settings - Fork 18
Meeting Minutes 2021 02 03
Brian Murphy edited this page Feb 3, 2021
·
1 revision
Meeting Date: 2021-02-03
Meeting Purpose: Regular catch-up
Note Taker: brainmurphy
- adammorr
- Alberto
- brainmurphy
- gareththackeray
- gkinsman
- james-fitzgerald-je
- martincostello
- maurofranchi
- slang25
| Item | Who | Notes |
|---|---|---|
| Setup a meeting | Brian | Done. Hello 👋🏻 |
| Item | Who | Notes |
|---|---|---|
| How to organise ourselves | Brian | We'll keep meeting minutes like this public. We'll use the GitHub Wiki to host them. |
| Review Project board | All | Tidied up several issues that were already done, moved some to backlog. In progress column looks better now. |
| Review open Pull Requests | All |
PR #845 (benchmarking tool) - OK to merge, reduction in code coverage is acceptable. PR #853 (logging namespace) - if we care enough to write this code, should we test it? Can MELT help? Comments on the PR, please. |
| PR #849 - TimeStampWithOffset | Gareth | Adding properties to the Message class is, generally, to be avoided. That said, recommendation (from MS, etc.) is to use DateTimeOffset to unambiguously identify a point in time. How could we safely migrate? Further comments on the PR, please. |
| Item | Who | Due Date |
|---|---|---|
| Merge George's benchmarking tool PR | Brian | |
| Further comments welcome on PR #849 | All |