[PR #604] [MERGED] Refactoring #617

Closed
opened 2026-02-26 04:35:47 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/mageddo/dns-proxy-server/pull/604
Author: @mageddo
Created: 11/18/2024
Status: Merged
Merged: 11/18/2024
Merged by: @mageddo

Base: developHead: feat/599-refactoring


📝 Commits (4)

📊 Changes

7 files changed (+114 additions, -59 deletions)

View changed files

📝 build.gradle (+2 -2)
📝 src/main/java/com/mageddo/dnsproxyserver/docker/dataprovider/ContainerFacade.java (+6 -2)
📝 src/main/java/com/mageddo/dnsproxyserver/docker/dataprovider/ContainerFacadeDefault.java (+19 -4)
📝 src/main/java/com/mageddo/dnsproxyserver/solver/docker/dataprovider/ContainerDAODefault.java (+2 -3)
📝 src/main/java/com/mageddo/dnsproxyserver/solver/docker/dataprovider/DpsContainerDAODefault.java (+1 -3)
📝 src/test/java/com/mageddo/dnsproxyserver/docker/dataprovider/ContainerFacadeDefaultCompTest.java (+7 -45)
src/test/java/com/mageddo/dnsproxyserver/docker/dataprovider/ContainerFacadeDefaultTest.java (+77 -0)

📄 Description

Refactoring changes made at #599


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/mageddo/dns-proxy-server/pull/604 **Author:** [@mageddo](https://github.com/mageddo) **Created:** 11/18/2024 **Status:** ✅ Merged **Merged:** 11/18/2024 **Merged by:** [@mageddo](https://github.com/mageddo) **Base:** `develop` ← **Head:** `feat/599-refactoring` --- ### 📝 Commits (4) - [`465e377`](https://github.com/mageddo/dns-proxy-server/commit/465e377425d4d99e80df1a176c3becb1d0a06ab7) revert - [`ef4a12f`](https://github.com/mageddo/dns-proxy-server/commit/ef4a12f147745a8d35c487ab7ff83771f150e7a7) refactoring - [`dba89ba`](https://github.com/mageddo/dns-proxy-server/commit/dba89ba3e81aaf3a865f28f9a664daf864f72b9f) refactoring test - [`53f9fb2`](https://github.com/mageddo/dns-proxy-server/commit/53f9fb25e6ed665939230bdbf36706d7a1a1b68f) breaking tests ### 📊 Changes **7 files changed** (+114 additions, -59 deletions) <details> <summary>View changed files</summary> 📝 `build.gradle` (+2 -2) 📝 `src/main/java/com/mageddo/dnsproxyserver/docker/dataprovider/ContainerFacade.java` (+6 -2) 📝 `src/main/java/com/mageddo/dnsproxyserver/docker/dataprovider/ContainerFacadeDefault.java` (+19 -4) 📝 `src/main/java/com/mageddo/dnsproxyserver/solver/docker/dataprovider/ContainerDAODefault.java` (+2 -3) 📝 `src/main/java/com/mageddo/dnsproxyserver/solver/docker/dataprovider/DpsContainerDAODefault.java` (+1 -3) 📝 `src/test/java/com/mageddo/dnsproxyserver/docker/dataprovider/ContainerFacadeDefaultCompTest.java` (+7 -45) ➕ `src/test/java/com/mageddo/dnsproxyserver/docker/dataprovider/ContainerFacadeDefaultTest.java` (+77 -0) </details> ### 📄 Description Refactoring changes made at #599 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 04:35:47 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/dns-proxy-server-mageddo#617
No description provided.