Calog timeline work - Githubissues

Shanefe commented 2 months ago

Changes

Working on getting better answers from Timeline.
- Added a couple of helper functions.
- Moved 2 of the large functions to their own functions (Let me know if they should be in their own files).
Matching Dev work on ShortClientInfoString.
Removed check on when getting large sets of Exception logs.

Shanefe commented 2 months ago

Seems like I should have gotten a notification of a failed build, but I didn't. I checked my GitHub settings, and it looks like I am subscribed for all alerts to go to email for this project.

Do you know if I should have gotten an alert to the spelling error failing build?

Shane

From: David Paulson @.> Sent: Thursday, May 2, 2024 11:25 AM To: microsoft/CSS-Exchange @.> Cc: Shane Ferrell @.>; Author @.> Subject: Re: [microsoft/CSS-Exchange] Calog timeline work (PR #2083)

@dpaulson45 commented on this pull request.

Looks like it should be fine. You just need to fix the spell check issue so the build pipeline passes.

— Reply to this email directly, view it on GitHubhttps://github.com/microsoft/CSS-Exchange/pull/2083#pullrequestreview-2036564191, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABPSHT3GHI4XXVZMM6WMSG3ZAKAJ3AVCNFSM6AAAAABHCRE2HSVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDAMZWGU3DIMJZGE. You are receiving this because you authored the thread.

dpaulson45 commented 2 months ago

I don't believe you do unless you kick off the build manually.

microsoft / CSS-Exchange

Calog timeline work #2083