Re: [PATCH] Re: Offline documentation (Org Manual info file) of org hooks is misleading

[libreville]

Great! Thanks for the help and nice working with you. :)

On 2023-06-21 11:58, Ihor Radchenko wrote:
> libreville  writes:
> 
>> Thanks a lot for the pointers.
>> Is this better?
> 
> Yup. Thanks!
> 
> Applied, onto main, with amendment.
> 
> I have removed
> 
>>  Date:  Wed Jun 21 13:04:19 2023 +0200
>>  Changes to be committed:
>>  modified:   doc/org-manual.org
> 
> I have also fixed the changelog entry and added "." at the end of the
> sentence there.
> 
> https://git.savannah.gnu.org/cgit/emacs/org-mode.git/commit/?id=eab92e8f7
> 
> You are now listed in the contributor list.
> https://git.sr.ht/~bzg/worg/commit/3f85dd4c

Re: [PATCH] Re: Offline documentation (Org Manual info file) of org hooks is misleading

[Ihor Radchenko]

libreville  writes:

> Thanks a lot for the pointers.
> Is this better?

Yup. Thanks!

Applied, onto main, with amendment.

I have removed

>  Date:  Wed Jun 21 13:04:19 2023 +0200
>  Changes to be committed:
>   modified:   doc/org-manual.org

I have also fixed the changelog entry and added "." at the end of the
sentence there.

https://git.savannah.gnu.org/cgit/emacs/org-mode.git/commit/?id=eab92e8f7

You are now listed in the contributor list.
https://git.sr.ht/~bzg/worg/commit/3f85dd4c

-- 
Ihor Radchenko // yantar92,
Org mode contributor,
Learn more about Org mode at .
Support Org development at ,
or support my work at 

[PATCH] Re: Offline documentation (Org Manual info file) of org hooks is misleading

[libreville]

Thanks a lot for the pointers.
Is this better?

On 2023-06-21 11:35, Ihor Radchenko wrote:
> libreville  writes:
> 
>> Here's my attempt.
> 
> Thanks!
> 
>> It's my first time doing this, so kindly let me know if there's
>> something I can do better!
> 
> Generally, we have instructions in 
> https://orgmode.org/worg/org-contribute.html#first-patch
> 
>> For example, should I have replaced the subject line of this email with
>> the subject in the formatted .patch?
> 
> Not necessarily. Although, adding [PATCH] to the subject would be
> helpful as an indication.
> 
>> From c1fda8d0162583db709c90f01df2b8678ddc7957 Mon Sep 17 00:00:00 2001
>> From: libreville 
>> Date: Wed, 21 Jun 2023 13:04:19 +0200
>> Subject: [PATCH] Delete reference to non-existent examples of hook usage.
> 
> 1. I'd add "org-manual: ..." to indicate which part of Org is modified.
> 2. There is no need to end the subject (first) line with "."
> 3. You need to add TINYCHANGE cookie, unless you have FSF copyright
>assignment.
> 4. Changelog entry would be good, although this is a simple patch and we
>can skip it here.
> 
> Hope it is going to be a good practice :)
From 6d0ade6af4ec4ab2fd8be578dd8851fae9e91253 Mon Sep 17 00:00:00 2001
From: libreville 
Date: Wed, 21 Jun 2023 13:04:19 +0200
Subject: [PATCH] org-manual.org: Delete ref to non-existent examples of hook
 usage

 Date:  Wed Jun 21 13:04:19 2023 +0200
 Changes to be committed:
	modified:   doc/org-manual.org

* org-manual.org:  Delete ref to non-existent examples of hook usage in
A.1 Hooks

TINYCHANGE
---
 doc/org-manual.org | 7 +++
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/doc/org-manual.org b/doc/org-manual.org
index 4feb15e71..21582fe8e 100644
--- a/doc/org-manual.org
+++ b/doc/org-manual.org
@@ -21128,10 +21128,9 @@ of Org.
 :END:
 #+cindex: hooks
 
-Org has a large number of hook variables for adding functionality.
-This appendix illustrates using a few.  A complete list of hooks with
-documentation is maintained by the Worg project at
-https://orgmode.org/worg/doc.html#hooks.
+Org has a large number of hook variables for adding functionality.  A
+complete list of hooks with documentation is maintained by the Worg
+project at https://orgmode.org/worg/doc.html#hooks.
 
 ** Add-on Packages
 :PROPERTIES:
-- 
2.30.2