Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add dns0.eu #2808

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add dns0.eu #2808

wants to merge 1 commit into from
+19 −1

Conversation

NixCookies
Copy link

@NixCookies NixCookies commented Sep 23, 2024 •
edited
Loading

Type of Change

  • New feature
  • Bug fix
  • Documentation update
  • Refactoring
  • Hotfix
  • Security patch
  • UI/UX improvement

Description

Added dns0.eu, a European public DNS service.
https://www.dns0.eu/

Testing

  • Successfully compiled using Compile.ps1 without issues.
  • Ran winutil.ps1 and was able to select and change the DNS settings.
  • Verified the settings in Windows.
  • Tested on Windows 11 IoT Enterprise LTSC 24H2.

Impact

  • Additional DNS Options: This change adds more flexibility for users by providing an additional public DNS service.
  • Regional Benefits: Specifically beneficial for users in European regions, offering a geographically closer DNS service.

Checklist

  • My code adheres to the coding and style guidelines of the project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no errors/warnings/merge conflicts.

Real-MullaC
Real-MullaC suggested changes Sep 24, 2024
View reviewed changes
Copy link
Contributor

@Real-MullaC Real-MullaC left a comment

Choose a reason for hiding this comment

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

Please remove this bit.

docs/dev/tweaks/z--Advanced-Tweaks---CAUTION/changedns.md Outdated Show resolved Hide resolved
ruxunderscore
ruxunderscore suggested changes Sep 24, 2024
View reviewed changes
Copy link
Contributor

@ruxunderscore ruxunderscore left a comment

Choose a reason for hiding this comment

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

As mentioned by Mulla, we just need changes to changedns.md removed, since this is an automatically generated file.

docs/dev/tweaks/z--Advanced-Tweaks---CAUTION/changedns.md Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ruxunderscore ruxunderscore ruxunderscore approved these changes

@Real-MullaC Real-MullaC Real-MullaC approved these changes

@ChrisTitusTech ChrisTitusTech Awaiting requested review from ChrisTitusTech ChrisTitusTech is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@NixCookies @ruxunderscore @Real-MullaC