Skip to content

Conversation

@zbalkan
Copy link

@zbalkan zbalkan commented Dec 17, 2025

Started small with minimal changes and tests. I'll create PRs project by project after this is accepted.

@zbalkan
Copy link
Author

zbalkan commented Dec 18, 2025

I started small with minimal changes. Whenever an issue is merged, I’ll work on the next one. Until we got 90% coverage, I'll proceed. Then, I'll go on with DnsServer.

@ShreyasZare
Copy link
Member

Thanks for the PR. Will review it soon.

@zbalkan
Copy link
Author

zbalkan commented Dec 27, 2025

I have several unit test branches waiting for this review.

@ShreyasZare
Copy link
Member

I have several unit test branches waiting for this review.

I need some time since there are too many outstanding issues. So please bear with the delay.

@zbalkan
Copy link
Author

zbalkan commented Dec 29, 2025

I am fine with the timing and delays as long as we are clear it is okay to develop unit tests for robustness. I would not like to spend time on these if you don't want to proceed that way.

@ShreyasZare
Copy link
Member

I am fine with the timing and delays as long as we are clear it is okay to develop unit tests for robustness. I would not like to spend time on these if you don't want to proceed that way.

If I had issue with unit tests then I would have already mentioned it.

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.

2 participants