Skip to content

Conversation

@Ndiritu
Copy link
Contributor

@Ndiritu Ndiritu commented Feb 5, 2025

@Ndiritu Ndiritu self-assigned this Feb 5, 2025
@Ndiritu Ndiritu force-pushed the feat/change-notification branch from cc166cc to 2316069 Compare February 11, 2025 21:50
@Ndiritu Ndiritu force-pushed the feat/change-notification branch from 2316069 to 3bfbb2f Compare February 11, 2025 21:51
@Ndiritu Ndiritu force-pushed the feat/change-notification branch from 97b380d to 5d36d3c Compare February 12, 2025 05:28
@Ndiritu Ndiritu marked this pull request as ready for review February 12, 2025 05:33
@Ndiritu Ndiritu requested a review from a team as a code owner February 12, 2025 05:33
andrueastman
andrueastman previously approved these changes Feb 12, 2025
@andrueastman
Copy link
Contributor

@baywet As I've made the latest updates here, any chance you can re-review before we merge?

@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@andrueastman andrueastman requested a review from baywet February 18, 2025 15:59
Copy link
Member

@baywet baywet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for making the changes!

@andrueastman andrueastman merged commit aa23bd7 into main Feb 18, 2025
11 of 12 checks passed
@andrueastman andrueastman deleted the feat/change-notification branch February 18, 2025 16:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants