Skip to content

🌟 [Major]: Add function for resolving DNS hosts#3

Merged
Marius Storhaug (MariusStorhaug) merged 8 commits intomainfrom
resolvehost
Aug 27, 2025
Merged

🌟 [Major]: Add function for resolving DNS hosts#3
Marius Storhaug (MariusStorhaug) merged 8 commits intomainfrom
resolvehost

Conversation

@MariusStorhaug
Copy link
Copy Markdown
Member

@MariusStorhaug Marius Storhaug (MariusStorhaug) commented Aug 27, 2025

Description

This pull request adds function for resolving DNS hosts by using the base .net class [System.Net.Dns].

Type of change

  • 📖 [Docs]
  • 🪲 [Fix]
  • 🩹 [Patch]
  • ⚠️ [Security fix]
  • 🚀 [Feature]
  • 🌟 [Breaking change]

Checklist

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas

…figuration, and type definitions, to streamline the module.
Copilot AI review requested due to automatic review settings August 27, 2025 09:00
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request is a major refactoring that removes most existing module content and introduces a DNS host resolution function. The change appears to be transitioning from a generic test module structure to a DNS-focused module.

  • Removes all existing test functions and module structure (variables, types, classes, etc.)
  • Adds Resolve-DnsHost function for DNS hostname resolution using .NET's System.Net.Dns class
  • Replaces existing test suite with DNS-specific tests

Reviewed Changes

Copilot reviewed 29 out of 30 changed files in this pull request and generated 2 comments.

File Description
src/functions/public/Resolve-DnsHost.ps1 New DNS resolution function using System.Net.Dns.Resolve()
tests/Dns.Tests.ps1 New test suite for DNS functionality
All other files Removal of existing module infrastructure and test content

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread src/functions/public/Resolve-DnsHost.ps1 Outdated
Comment thread src/functions/public/Resolve-DnsHost.ps1 Outdated
@MariusStorhaug Marius Storhaug (MariusStorhaug) merged commit c6d774c into main Aug 27, 2025
34 of 36 checks passed
@github-actions
Copy link
Copy Markdown

Module Dns - 1.0.0 published to the PowerShell Gallery.

@github-actions
Copy link
Copy Markdown

GitHub release for Dns v1.0.0 has been created.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🩹 [Patch]: Add function to resolve DNS

2 participants