Skip to content

Commit fd2bffa

Browse files
authored
Merge pull request #231 from avinab-neogy/docs/220-patch-naming
Update patch naming documentation
2 parents 4e99cf0 + 7fc24fb commit fd2bffa

File tree

3 files changed

+14
-7
lines changed

3 files changed

+14
-7
lines changed

.github/workflows/build-mkdocs-website.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ on:
1212

1313
permissions:
1414
contents: write
15-
15+
1616
jobs:
1717
build-and-deploy:
1818
if: |

INIT

Lines changed: 0 additions & 3 deletions
This file was deleted.

docs/tutorials/patch_update.md

Lines changed: 13 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,14 +14,24 @@ this first.
1414

1515
Go to the source directory and use `svn diff` to create a patch.
1616

17+
**Use a descriptive patch name with the bug number and a short description,
18+
rather than a generic name like `patch.diff`.**
19+
1720
```bash
1821
cd $TOP_SRCDIR
19-
svn diff > $PATCHDIR/patch.diff
22+
svn diff > $PATCHDIR/16629-infinite-recursion.diff
2023
```
2124

25+
The example above uses `16629-infinite-recursion.diff` - this name follows the
26+
convention: bug number (16629), short description (infinite-recursion), and
27+
`.diff` extension, making patches easy to identify and review.
28+
2229
The patch file will be saved in the directory specified by the PATCHDIR
23-
environment variable that is defined when the codespace starts
30+
environment variable that is defined when the codespace starts.
2431

2532
```bash
26-
echo $PATCHDIR/patch.diff
33+
ls $PATCHDIR
2734
```
35+
36+
lists all patch files in your patch directory, allowing you to easily
37+
see and verify the patch files you have created.

0 commit comments

Comments
 (0)