Skip to content

Some corrections #2127

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Some corrections #2127

wants to merge 4 commits into from

Conversation

zwJimRaynor
Copy link
Contributor

No description provided.

Copy link
Contributor

Learn Build status updates of commit 2725892:

⚠️ Validation status: warnings

File Status Preview URL Details
WSL/build-custom-distro.md ⚠️Warning View Details
WSL/case-sensitivity.md ⚠️Warning View Details
WSL/enterprise.md ⚠️Warning View Details
WSL/faq.yml ⚠️Warning View Details
WSL/file-permissions.md ⚠️Warning View Details
WSL/use-custom-distro.md ⚠️Warning View Details
WSL/install-manual.md 💡Suggestion View Details
WSL/about.md ✅Succeeded View
WSL/basic-commands.md ✅Succeeded View
WSL/connect-usb.md ✅Succeeded View
WSL/disk-space.md ✅Succeeded View
WSL/docfx.json ✅Succeeded
WSL/filesystems.md ✅Succeeded View
WSL/index.md ✅Succeeded View
WSL/install-on-server.md ✅Succeeded View
WSL/install.md ✅Succeeded View
WSL/intune.md ✅Succeeded View
WSL/media/store.png ✅Succeeded View
WSL/media/UbuntuStore.png ✅Succeeded View
WSL/media/wsl-install.png ✅Succeeded n/a (file deleted or renamed)
WSL/media/wsl-settings.png ✅Succeeded View
WSL/networking.md ✅Succeeded View
WSL/release-notes.md ✅Succeeded View
WSL/setup/environment.md ✅Succeeded View
WSL/toc.yml ✅Succeeded View

This comment lists only the first 25 files in the pull request.

WSL/build-custom-distro.md

  • Line 35, Column 138: [Warning: bookmark-not-found - See documentation] Cannot find bookmark '#what-is-the-difference-between-wslconf-and-wslconfig' in 'wsl-config.md'.

WSL/case-sensitivity.md

  • Line 115, Column 43: [Warning: bookmark-not-found - See documentation] Cannot find bookmark '#configure-settings-with-wslconfig-and-wslconf' in 'wsl-config.md'.

WSL/enterprise.md

  • Line 43, Column 40: [Warning: bookmark-not-found - See documentation] Cannot find bookmark '#configuration-setting-for-wslconfig' in 'wsl-config.md', did you mean '#configuration-settings-for-wslconfig'?

WSL/faq.yml

  • Line 216, Column 88: [Warning: disallowed-html-tag - See documentation] HTML tag 'name' isn't allowed. Replace it with approved Markdown or escape the brackets if the content is a placeholder.

WSL/file-permissions.md

  • Line 104, Column 13: [Warning: bookmark-not-found - See documentation] Cannot find bookmark '#per-distribution-configuration-options-with-wslconf' in 'wsl-config.md'.

WSL/use-custom-distro.md

  • Line 129, Column 51: [Warning: bookmark-not-found - See documentation] Cannot find bookmark '#configure-settings-with-wslconfig-and-wslconf' in 'wsl-config.md'.

WSL/install-manual.md

  • Line 204, Column 5: [Suggestion: other-site-link-broken - See documentation] Link 'https://publicwsldistros.blob.core.windows.net/wsldistrostorage/OracleLinux_8.7-230428.Appx"' points to a page that doesn't exist. Check the path or URL and update the link.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

Copy link
Contributor

Learn Build status updates of commit 1ad7e05:

⚠️ Validation status: warnings

File Status Preview URL Details
WSL/faq.yml ⚠️Warning View Details
WSL/about.md ✅Succeeded View
WSL/basic-commands.md ✅Succeeded View
WSL/build-custom-distro.md ✅Succeeded View
WSL/case-sensitivity.md ✅Succeeded View
WSL/connect-usb.md ✅Succeeded View
WSL/disk-space.md ✅Succeeded View
WSL/docfx.json ✅Succeeded
WSL/enterprise.md ✅Succeeded View
WSL/file-permissions.md ✅Succeeded View
WSL/filesystems.md ✅Succeeded View
WSL/index.md ✅Succeeded View
WSL/install-manual.md ✅Succeeded View
WSL/install-on-server.md ✅Succeeded View
WSL/install.md ✅Succeeded View
WSL/intune.md ✅Succeeded View
WSL/media/store.png ✅Succeeded View
WSL/media/UbuntuStore.png ✅Succeeded View
WSL/media/wsl-install.png ✅Succeeded n/a (file deleted or renamed)
WSL/media/wsl-settings.png ✅Succeeded View
WSL/networking.md ✅Succeeded View
WSL/release-notes.md ✅Succeeded View
WSL/setup/environment.md ✅Succeeded View
WSL/toc.yml ✅Succeeded View
WSL/troubleshooting.md ✅Succeeded View

This comment lists only the first 25 files in the pull request.

WSL/faq.yml

  • Line 216, Column 88: [Warning: disallowed-html-tag - See documentation] HTML tag 'name' isn't allowed. Replace it with approved Markdown or escape the brackets if the content is a placeholder.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

Copy link
Contributor

Learn Build status updates of commit efb5b54:

⚠️ Validation status: warnings

File Status Preview URL Details
WSL/faq.yml ⚠️Warning View Details
WSL/about.md ✅Succeeded View
WSL/basic-commands.md ✅Succeeded View
WSL/build-custom-distro.md ✅Succeeded View
WSL/case-sensitivity.md ✅Succeeded View
WSL/connect-usb.md ✅Succeeded View
WSL/disk-space.md ✅Succeeded View
WSL/docfx.json ✅Succeeded
WSL/enterprise.md ✅Succeeded View
WSL/file-permissions.md ✅Succeeded View
WSL/filesystems.md ✅Succeeded View
WSL/index.md ✅Succeeded View
WSL/install-manual.md ✅Succeeded View
WSL/install-on-server.md ✅Succeeded View
WSL/install.md ✅Succeeded View
WSL/intune.md ✅Succeeded View
WSL/media/store.png ✅Succeeded View
WSL/media/UbuntuStore.png ✅Succeeded View
WSL/media/wsl-install.png ✅Succeeded n/a (file deleted or renamed)
WSL/media/wsl-settings.png ✅Succeeded View
WSL/networking.md ✅Succeeded View
WSL/release-notes.md ✅Succeeded View
WSL/setup/environment.md ✅Succeeded View
WSL/toc.yml ✅Succeeded View
WSL/troubleshooting.md ✅Succeeded View

This comment lists only the first 25 files in the pull request.

WSL/faq.yml

  • Line 216, Column 88: [Warning: disallowed-html-tag - See documentation] HTML tag 'name' isn't allowed. Replace it with approved Markdown or escape the brackets if the content is a placeholder.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

Copy link
Contributor

Learn Build status updates of commit 9b6bbc1:

⚠️ Validation status: warnings

File Status Preview URL Details
WSL/faq.yml ⚠️Warning View Details
WSL/tutorials/wsl-git.md 💡Suggestion View Details
WSL/about.md ✅Succeeded View
WSL/basic-commands.md ✅Succeeded View
WSL/build-custom-distro.md ✅Succeeded View
WSL/case-sensitivity.md ✅Succeeded View
WSL/connect-usb.md ✅Succeeded View
WSL/disk-space.md ✅Succeeded View
WSL/docfx.json ✅Succeeded
WSL/enterprise.md ✅Succeeded View
WSL/file-permissions.md ✅Succeeded View
WSL/filesystems.md ✅Succeeded View
WSL/index.md ✅Succeeded View
WSL/install-manual.md ✅Succeeded View
WSL/install-on-server.md ✅Succeeded View
WSL/install.md ✅Succeeded View
WSL/intune.md ✅Succeeded View
WSL/media/store.png ✅Succeeded View
WSL/media/UbuntuStore.png ✅Succeeded View
WSL/media/wsl-install.png ✅Succeeded n/a (file deleted or renamed)
WSL/media/wsl-settings.png ✅Succeeded View
WSL/networking.md ✅Succeeded View
WSL/release-notes.md ✅Succeeded View
WSL/setup/environment.md ✅Succeeded View
WSL/toc.yml ✅Succeeded View

This comment lists only the first 25 files in the pull request.

WSL/faq.yml

  • Line 216, Column 88: [Warning: disallowed-html-tag - See documentation] HTML tag 'name' isn't allowed. Replace it with approved Markdown or escape the brackets if the content is a placeholder.

WSL/tutorials/wsl-git.md

  • Line 66, Column 270: [Suggestion: other-site-link-broken - See documentation] Link 'https://dev.azure.com/' points to a page that doesn't exist. Check the path or URL and update the link.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

Copy link
Contributor

Learn Build status updates of commit afa7dc7:

⚠️ Validation status: warnings

File Status Preview URL Details
WSL/faq.yml ⚠️Warning View Details
WSL/tutorials/wsl-git.md 💡Suggestion View Details
WSL/about.md ✅Succeeded View
WSL/basic-commands.md ✅Succeeded View
WSL/build-custom-distro.md ✅Succeeded View
WSL/case-sensitivity.md ✅Succeeded View
WSL/connect-usb.md ✅Succeeded View
WSL/disk-space.md ✅Succeeded View
WSL/docfx.json ✅Succeeded
WSL/enterprise.md ✅Succeeded View
WSL/file-permissions.md ✅Succeeded View
WSL/filesystems.md ✅Succeeded View
WSL/index.md ✅Succeeded View
WSL/install-manual.md ✅Succeeded View
WSL/install-on-server.md ✅Succeeded View
WSL/install.md ✅Succeeded View
WSL/intune.md ✅Succeeded View
WSL/media/store.png ✅Succeeded View
WSL/media/UbuntuStore.png ✅Succeeded View
WSL/media/wsl-install.png ✅Succeeded n/a (file deleted or renamed)
WSL/media/wsl-settings.png ✅Succeeded View
WSL/networking.md ✅Succeeded View
WSL/release-notes.md ✅Succeeded View
WSL/setup/environment.md ✅Succeeded View
WSL/toc.yml ✅Succeeded View

This comment lists only the first 25 files in the pull request.

WSL/faq.yml

  • Line 216, Column 88: [Warning: disallowed-html-tag - See documentation] HTML tag 'name' isn't allowed. Replace it with approved Markdown or escape the brackets if the content is a placeholder.

WSL/tutorials/wsl-git.md

  • Line 66, Column 270: [Suggestion: other-site-link-broken - See documentation] Link 'https://dev.azure.com' points to a page that doesn't exist. Check the path or URL and update the link.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

Copy link
Contributor

Learn Build status updates of commit 93e8600:

⚠️ Validation status: warnings

File Status Preview URL Details
WSL/faq.yml ⚠️Warning View Details
WSL/about.md ✅Succeeded View
WSL/basic-commands.md ✅Succeeded View
WSL/build-custom-distro.md ✅Succeeded View
WSL/case-sensitivity.md ✅Succeeded View
WSL/connect-usb.md ✅Succeeded View
WSL/disk-space.md ✅Succeeded View
WSL/docfx.json ✅Succeeded
WSL/enterprise.md ✅Succeeded View
WSL/file-permissions.md ✅Succeeded View
WSL/filesystems.md ✅Succeeded View
WSL/index.md ✅Succeeded View
WSL/install-manual.md ✅Succeeded View
WSL/install-on-server.md ✅Succeeded View
WSL/install.md ✅Succeeded View
WSL/intune.md ✅Succeeded View
WSL/media/appstorage-11.png ✅Succeeded View
WSL/media/firewallrulemerging.png ✅Succeeded View
WSL/media/ghost_adapter_1.png ✅Succeeded View
WSL/media/ghost_adapter_2.png ✅Succeeded View
WSL/media/ghost_adapter_3.png ✅Succeeded View
WSL/media/localappdata.png ✅Succeeded View
WSL/media/ntfs-properties.png ✅Succeeded View
WSL/media/store.png ✅Succeeded View
WSL/media/system.png ✅Succeeded View

This comment lists only the first 25 files in the pull request.

WSL/faq.yml

  • Line 217, Column 98: [Warning: disallowed-html-tag - See documentation] HTML tag 'name' isn't allowed. Replace it with approved Markdown or escape the brackets if the content is a placeholder.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

Copy link
Contributor

Learn Build status updates of commit 8550ac6:

⚠️ Validation status: warnings

File Status Preview URL Details
WSL/faq.yml ⚠️Warning View Details
WSL/about.md ✅Succeeded View
WSL/basic-commands.md ✅Succeeded View
WSL/build-custom-distro.md ✅Succeeded View
WSL/case-sensitivity.md ✅Succeeded View
WSL/connect-usb.md ✅Succeeded View
WSL/disk-space.md ✅Succeeded View
WSL/docfx.json ✅Succeeded
WSL/enterprise.md ✅Succeeded View
WSL/file-permissions.md ✅Succeeded View
WSL/filesystems.md ✅Succeeded View
WSL/index.md ✅Succeeded View
WSL/install-manual.md ✅Succeeded View
WSL/install-on-server.md ✅Succeeded View
WSL/install.md ✅Succeeded View
WSL/intune.md ✅Succeeded View
WSL/media/appstorage-11.png ✅Succeeded View
WSL/media/firewallrulemerging.png ✅Succeeded View
WSL/media/ghost_adapter_1.png ✅Succeeded View
WSL/media/ghost_adapter_2.png ✅Succeeded View
WSL/media/ghost_adapter_3.png ✅Succeeded View
WSL/media/localappdata.png ✅Succeeded View
WSL/media/ntfs-properties.png ✅Succeeded View
WSL/media/store.png ✅Succeeded View
WSL/media/system.png ✅Succeeded View

This comment lists only the first 25 files in the pull request.

WSL/faq.yml

  • Line 211, Column 98: [Warning: disallowed-html-tag - See documentation] HTML tag 'name' isn't allowed. Replace it with approved Markdown or escape the brackets if the content is a placeholder.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

Copy link
Contributor

Learn Build status updates of commit 3d7f242:

✅ Validation status: passed

File Status Preview URL Details
WSL/about.md ✅Succeeded View
WSL/basic-commands.md ✅Succeeded View
WSL/build-custom-distro.md ✅Succeeded View
WSL/case-sensitivity.md ✅Succeeded View
WSL/connect-usb.md ✅Succeeded View
WSL/disk-space.md ✅Succeeded View
WSL/docfx.json ✅Succeeded
WSL/enterprise.md ✅Succeeded View
WSL/faq.yml ✅Succeeded View
WSL/file-permissions.md ✅Succeeded View
WSL/filesystems.md ✅Succeeded View
WSL/index.md ✅Succeeded View
WSL/install-manual.md ✅Succeeded View
WSL/install-on-server.md ✅Succeeded View
WSL/install.md ✅Succeeded View
WSL/intune.md ✅Succeeded View
WSL/media/appstorage-11.png ✅Succeeded View
WSL/media/firewallrulemerging.png ✅Succeeded View
WSL/media/ghost_adapter_1.png ✅Succeeded View
WSL/media/ghost_adapter_2.png ✅Succeeded View
WSL/media/ghost_adapter_3.png ✅Succeeded View
WSL/media/localappdata.png ✅Succeeded View
WSL/media/ntfs-properties.png ✅Succeeded View
WSL/media/store.png ✅Succeeded View
WSL/media/system.png ✅Succeeded View

This comment lists only the first 25 files in the pull request.
For more details, please refer to the build report.

For any questions, please:

@mattwojo
Copy link
Collaborator

Hey @zwJimRaynor - Thank you for this PR and all of the time and attention that went into it! We just looked it over and there is a lot of good updates. The issue is that it's too much in a single PR and trying to accomplish too many things for us to accept and merge it all in one PR.
Would you be able to break this up a bit into more targeted PRs... for example, all of the link fixes in one PR, but paragraph / content rewrites in a different PR? I'm sorry to ask you to do this, but we are having too much trouble merging as-is.
There are some great tips about creating smaller PRs here: https://gist.github.com/sktse/569cb192ce1518f83db58567591e3205

Please let me know what you think and if this seems doable.
Thanks again!

@zwJimRaynor
Copy link
Contributor Author

Please give me some time and I will resolve the merge conflicts and optimize the commit log. Because the Windows version I'm currently using cannot properly use Hyper-V, I am temporarily unable to continue writing the documentation.🫠

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