Skip to content

Conversation

@Sharath-a26
Copy link

  • Created the shared logger using zero log. Followed the same approach used in auth_microservice.

@Ashrockzzz2003 Please review and let me know if there if any changes

@Ashrockzzz2003 Ashrockzzz2003 self-requested a review October 31, 2025 05:58
@Ashrockzzz2003 Ashrockzzz2003 linked an issue Oct 31, 2025 that may be closed by this pull request
Copy link
Member

@Ashrockzzz2003 Ashrockzzz2003 left a comment

Choose a reason for hiding this comment

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

@Sharath-a26 please resolve merge conflicts. Just merged a small important PR today.

@Sharath-a26
Copy link
Author

Okay, will do it

Copy link
Author

@Sharath-a26 Sharath-a26 left a comment

Choose a reason for hiding this comment

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

Merged all conflicts

@Ashrockzzz2003
Copy link
Member

@Sharath-a26

I think when you resolved conflicts
you accidentally included out of date changes too
Like amqp was completely removed, but still is there in this PR

@Ashrockzzz2003
Copy link
Member

Closing this PR rn as its hard to resolve

If you still wanna fix, please do and reopen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use a shared logger.

2 participants