Skip to content

Conversation

@BornToBeRoot
Copy link
Owner

Changes proposed in this pull request

  • DNS Lookup - Add MX priority

Related issue(s)

Copilot generated summary

Provide a Copilot generated summary of the changes in this pull request.

Copilot summary

This pull request mainly includes code cleanup and improvements to the formatting and consistency of DNS record processing in the DNSLookup class. It also updates the user dictionary for spell checking. The most important changes are grouped below:

Code cleanup and formatting:

  • Removed an unused using System.Diagnostics; directive from DNSLookup.cs to clean up the imports.
  • Improved line breaks and formatting in the ProcessDnsAnswers method to enhance readability when constructing DNSLookupRecordInfo for various DNS record types. [1] [2] [3]
  • Removed unnecessary blank lines and made minor comment clarifications in DNSLookup.cs. [1] [2] [3]

Spell check dictionary update:

  • Added dnskey to the user dictionary in NETworkManager.sln.DotSettings to prevent spell check warnings for this term.

To-Do

Contributing

By submitting this pull request, I confirm the following:

@mergify mergify bot merged commit 52d0c5c into main Aug 5, 2025
4 checks passed
@mergify mergify bot deleted the fix/dnslookup_mx_prio branch August 5, 2025 19:42
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.

DNS Lookup - MX Record doesn't output priotity

2 participants