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

wiki: openwrt naming fixes and contribution improvement #25731

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

Conversation

GrumpyMeow
Copy link
Contributor

@GrumpyMeow GrumpyMeow commented Jan 13, 2025

Maintainer: various

Description:
With this PR i:

  • correctly named the reference to a menu-item
  • made the contribution text somewhat clearer
  • fix openwrt reference with correct capitalization
  • fix some typo's in package markdown files

Made this PR with best intentions. No problem if unable to complete this PR.

@GrumpyMeow GrumpyMeow marked this pull request as ready for review January 13, 2025 10:41
@GrumpyMeow GrumpyMeow changed the title wiki markdown: openwrt naming fixes and contribution improvement wiki: openwrt naming fixes and contribution improvement Jan 13, 2025
@efahl
Copy link
Contributor

efahl commented Jan 13, 2025

fix openwrt reference with correct capitalization

Haha, my pet peeve! Thanks for that one.

Looks good to me.

@GrumpyMeow GrumpyMeow force-pushed the feature_markdown_docs_improvement branch from a6e8e19 to 13291b6 Compare January 13, 2025 19:38
@GrumpyMeow GrumpyMeow force-pushed the feature_markdown_docs_improvement branch from 13bbd5d to 9782b35 Compare January 13, 2025 19:40
Copy link
Member

@1715173329 1715173329 left a comment

Choose a reason for hiding this comment

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

thanks for the "OpenWrt" spell correction, my pet peeve too.

"General Build Settings" set if language support is relevant to your package.
- Best of all -- it works as expected!
- The compilation has been successfully tested with the appropriate includes and dependencies.
Additionally, if language support is relevant to your package, please enable the "Compile with full language support" option found under "Global Build Settings" in "menuconfig" and conduct further tests.
Copy link
Member

Choose a reason for hiding this comment

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

this looks pretty long, would be better to wrap a new line

- <strong>PKG_SOURCE</strong> should reference the smallest available archive. In order of
preference:
1. xz (most compressed),
1. bzip2, gz and zip.
Copy link
Member

Choose a reason for hiding this comment

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

should be 2. here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Markdown does the numbering automatically upon viewing. So I left this on-purpose like this.

Copy link
Member

Choose a reason for hiding this comment

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

yep but this is confusing when review the source code

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.

3 participants