From: Andrea Cervesato via ltp <ltp@lists.linux.it>
To: Cyril Hrubis <chrubis@suse.cz>,
Andrea Cervesato <andrea.cervesato@suse.de>
Cc: ltp@lists.linux.it
Subject: Re: [LTP] [PATCH v2] New LTP documentation
Date: Thu, 4 Apr 2024 15:07:13 +0200 [thread overview]
Message-ID: <d6a87d99-923b-49c9-abe1-a7d37e9f43eb@suse.com> (raw)
In-Reply-To: <Zg6lbcX3_8LYj3k6@yuki>
Hi,
On 4/4/24 15:04, Cyril Hrubis wrote:
> Hi!
> Can we please fix the spelling errors before this patchset is applied?
>
> $ make spelling
> ...
> WARNING: developers/build_system.rst:134: : Spell check: Explaination: Explaination.
> WARNING: developers/test_case_tutorial.rst:136: : Spell check: initialisation: initialisation code. It is important to note that this is a completely.
> WARNING: developers/test_case_tutorial.rst:378: : Spell check: internalising: ABI (Application Binary Interface), rather than internalising actual.
> WARNING: developers/test_case_tutorial.rst:638: : Spell check: dependants: dependants before their dependencies..
> WARNING: developers/test_case_tutorial.rst:663: : Spell check: organisational: We are about to make some organisational changes to the test, so now would be.
> WARNING: developers/test_case_tutorial.rst:1004: : Spell check: organisational: Style and organisational issues are best left to after you have found logical.
> WARNING: developers/test_case_tutorial.rst:1030: : Spell check: synchronising: also various helper functions for synchronising and forking processes..
> WARNING: developers/writing_tests.rst:51: : Spell check: overcomment: Do not overcomment.
> WARNING: developers/writing_tests.rst:67: : Spell check: highlevel: paragraph with highlevel test description at the beginning of the file right.
> WARNING: developers/writing_tests.rst:89: : Spell check: vendored: uses (among other checks) our vendored version of.
> WARNING: developers/writing_tests.rst:162: : Spell check: vendored: It uses (among other checks) our vendored version of.
> WARNING: developers/writing_tests.rst:303: : Spell check: Subexecutables: Subexecutables.
> WARNING: maintainers/patch_review.rst:29: : Spell check: behaviour: Use sanitizers e.g. undefined behaviour, address..
> WARNING: maintainers/patch_review.rst:66: : Spell check: behaviour: Could future kernel behaviour change without "breaking userland"?.
> WARNING: maintainers/patch_review.rst:141: : Spell check: licence: ; the licence for test (e.g. GPL-2.0) should not change.
> WARNING: users/setup_tests.rst:7: : Spell check: behaviour: tests behaviour by setting environment variables and using specific tests.
> WARNING: users/setup_tests.rst:38: : Spell check: behaviour: file. Force colorized output behaviour: .
> WARNING: users/setup_tests.rst:64: : Spell check: tella: empty string, tella the library that system is not a virtual machine..
> WARNING: users/setup_tests.rst:69: : Spell check: addjust: It's required to addjust path: .
> WARNING: users/setup_tests.rst:97: : Spell check: commandline: variable or set on a commandline by the .
> WARNING: users/setup_tests.rst:96: : Spell check: Hoewever: parameter. Hoewever,.
> WARNING: Found 21 misspelled words
> build succeeded, 22 warnings.
>
> Most of these are actually valid typos.
>
> Other than that:
>
> Reviewed-by: Cyril Hrubis <chrubis@suse.cz>
>
Sure, good idea.
Andrea
--
Mailing list info: https://lists.linux.it/listinfo/ltp
prev parent reply other threads:[~2024-04-04 13:07 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-04-04 12:15 [LTP] [PATCH v2] New LTP documentation Andrea Cervesato
2024-04-04 12:24 ` Andrea Cervesato via ltp
2024-04-04 12:33 ` Petr Vorel
2024-04-04 13:04 ` Cyril Hrubis
2024-04-04 13:07 ` Andrea Cervesato via ltp [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=d6a87d99-923b-49c9-abe1-a7d37e9f43eb@suse.com \
--to=ltp@lists.linux.it \
--cc=andrea.cervesato@suse.com \
--cc=andrea.cervesato@suse.de \
--cc=chrubis@suse.cz \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).