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.

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

Improved wireguard_vpn.md quality and readability #2399

Merged
merged 1 commit into from
Sep 29, 2024

Conversation

j0eybrinkman
Copy link
Contributor

@j0eybrinkman j0eybrinkman commented Sep 29, 2024

  • Fixed a few typos
  • Added -y to commands so user input isn't required
  • Changed public/private key pair naming scheme to interface name (less confusing)
  • Rearranged and re-worded some content for better readability
  • Changed editor to vi since vim isn't always installed by default
  • Added instructions on permanently adding peers to interface

Author checklist (Completed by original Author)

  • Good fit for the Rocky Linux project? Title and Author Metatags inserted ?
  • If applicable, steps and instructions have been tested to work
  • Initial self-review to fix basic typos and grammar completed

Rocky Documentation checklist (Completed by Rocky team)

  • 1st Pass (Document is good fit for project and author checklist completed)
  • 2nd Pass (Technical Review - check for technical correctness)
  • 3rd Pass (Detailed Editorial Review and Peer Review)
  • Final approval (Final Review)

Copy link

Test results for 1848fa0:

Number of broken URLs: 4

URL,RESULT,FILENAME
 http://www.example.org/foo.txt,failed,books/web_services/052-load-balancer-proxies-varnish.md
 https://azuremarketplace.microsoft.com/en-us/marketplace/apps/resf.rockylinux-aarch64,failed,guides/cloud/migration-to-new-azure-images.md
 https://kb.vmware.com/s/article/71264,failed,guides/automation/templates-automation-packer-vsphere.md
 https://www.winehq.org/,failed,desktop/gaming/proton.md

@sspencerwire sspencerwire merged commit 21ba618 into rocky-linux:main Sep 29, 2024
3 checks passed
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