Skip to content

Commit ceddaf6

Browse files
Stypoxopusforlife2
andauthored
Fix grammar and wording as suggested in review
Co-authored-by: opusforlife2 <53176348+opusforlife2@users.noreply.github.com>
1 parent 5a57e6c commit ceddaf6

File tree

1 file changed

+23
-23
lines changed

1 file changed

+23
-23
lines changed

docs/NaN_Full_release_docs.md

Lines changed: 23 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -8,24 +8,24 @@
88

99
#### Repositories
1010

11-
- have a cloned NewPipe local repository (for the rest of the page `origin` is assumed to be the remote at `github.com/TeamNewPipe/NewPipe`)
12-
- add the `weblate` remote to the same local repository (the URL used below can be found in the Maintenance page on Weblate)
11+
- have a cloned NewPipe local repository (for the rest of the page, `origin` is assumed to be the remote at `github.com/TeamNewPipe/NewPipe`)
12+
- add the `weblate` remote to the same local repository (the URL used below can be found on the Maintenance page on Weblate)
1313
- `git remote add weblate https://hosted.weblate.org/git/newpipe/strings/`
1414
- make sure there are no pending changes
1515
- `git clean -fdx` to **discard** them all (**CAUTION**)
16-
- go on the `dev` branch and make sure is up-to-date with remote:
16+
- switch to the `dev` branch and make sure it is up-to-date with the remote:
1717
- `git checkout dev`
1818
- `git pull origin dev`
1919

2020
#### Version name and conventions
2121

22-
- find the version code of the next release by looking for `versionCode` into [`app/build.gradle`](https://github.com/TeamNewPipe/NewPipe/blob/dev/app/build.gradle): that value plus one (from now on called `NEW_VERSION_CODE`) will be the new value (but do not edit the file yet)
22+
- find the version code of the next release by looking for `versionCode` in [`app/build.gradle`](https://github.com/TeamNewPipe/NewPipe/blob/dev/app/build.gradle): 1 added to that value (from now on called `NEW_VERSION_CODE`) will be the new value (but do not edit the file yet)
2323
- choose the version number of the next release according to [semantic versioning](http://semver.org/) (from now on called `X.X.X`)
2424

2525
#### Identification
2626

2727
- have `gpg` installed and usable on your PC
28-
- have a GPG key to be used to make sure it is actually you
28+
- have a GPG key, which can be used to verify that a file is really from you
2929

3030
### Pull changes from Weblate
3131

@@ -53,10 +53,10 @@
5353
### Create a changelog
5454
5555
- finalize the draft changelog [kept on GitHub](https://github.com/TeamNewPipe/NewPipe/releases), in case there are still some things to fill in
56-
- remove temporary instructions and numbers before `-`, that keep track in which order the PRs were merged, but which is useful information only for blogpost writers
56+
- remove the temporary instructions, and the numbers before `-` which keep track of the order in which the PRs were merged, as that info is useful only for the blog post writers
5757
- before removing that information, you may want to send the original changelog to the blogpost writers
5858
- create a new English changelog in the [`fastlane/metadata/android/en-US/changelogs/`](https://github.com/TeamNewPipe/NewPipe/blob/dev/fastlane/metadata/android/en-US/changelogs/) folder
59-
- the file should be named `NEW_VERSION_CODE.txt`, using the new version code found in [Preliminary steps](#preliminary-steps)
59+
- the file should be named `NEW_VERSION_CODE.txt`, using the new version code found in the [Preliminary steps](#preliminary-steps)
6060
- the file should have this structure (sections with no points can be removed):
6161
```
6262
New
@@ -80,15 +80,15 @@
8080
8181
### Push the changelog to Weblate
8282
83-
Now there shuold be two new commits (the Weblate and changelog ones) on your local `dev` branch which are not on NewPipe's remote `dev` branch.
83+
Now there should be two new commits (the Weblate and changelog ones) on your local `dev` branch, which are not on NewPipe's remote `dev` branch.
8484
- if you are an admin of the NewPipe repo, just push the changes to the remote `dev`
8585
- `git push origin dev`
8686
- if you are not an admin, create a pull request normally and ask someone with maintainer access to merge it
8787
- go to [Weblate's Maintenance tab](https://hosted.weblate.org/projects/newpipe/#repository)
8888
- press the *Update* button to update Weblate with the commit you just pushed on NewPipe's `dev` branch
8989
- **press the *Unlock*** button to allow translators to translate the changelog and possibly other components (**do not forget this step!**)
9090
- note that we had to do this process on NewPipe's `dev` branch because:
91-
- Weblate's components are connected to NewPipe's `dev` branch and will update changes from there
91+
- Weblate's components are connected to NewPipe's `dev` branch, and will update changes from there
9292
- Weblate's git repo is not writable, so there is no way to push commits there manually
9393
9494
### Creating the release branch
@@ -97,11 +97,11 @@ Now there shuold be two new commits (the Weblate and changelog ones) on your loc
9797
- `git checkout -b release-X.X.X`
9898
- edit the [`app/build.gradle`](https://github.com/TeamNewPipe/NewPipe/blob/dev/app/build.gradle) file to update the extractor
9999
- set the NewPipeExtractor dependency version to a suitable one (usually [the last commit in the NewPipeExtractor repo](https://github.com/TeamNewPipe/NewPipeExtractor/commits/dev))
100-
- commit the extractor update (if you used a specific version, specify `to VERSION` in the commit message)
100+
- commit the extractor update (if you used a specific version, append `to VERSION` to the commit message)
101101
- `git add app/build.gradle`
102102
- `git commit -m "Update NewPipeExtractor"`
103103
- edit the [`app/build.gradle`](https://github.com/TeamNewPipe/NewPipe/blob/dev/app/build.gradle) file to bump the release
104-
- set `versionCode` to `NEW_VERSION_CODE`, i.e. increment the value by one as described in [Preliminary steps](#preliminary-steps)
104+
- set `versionCode` to `NEW_VERSION_CODE`, i.e. increment the value by 1 as described in the [Preliminary steps](#preliminary-steps)
105105
- set `versionName` to `"X.X.X"`
106106
- commit the version bump (try to stick to the provided commit message template)
107107
- `git add app/build.gradle`
@@ -115,14 +115,14 @@ Now there shuold be two new commits (the Weblate and changelog ones) on your loc
115115
- if you used the correct branch name you should be able to use this url, after changing the X.X.X: https://github.com/TeamNewPipe/NewPipe/pull/new/release-X.X.X
116116
- make sure the PR has `master` as the *base* branch and `release-X.X.X` as the *compare* branch
117117
- the PR title should be "Release vX.X.X (NEW_VERSION_CODE)"
118-
- remove all of the PR template and instead put these two lines in the description (the `ISSUE_NUMBER` will be replaced later):
118+
- remove the entire PR template, and instead put these two lines in the description (the `ISSUE_NUMBER` will be replaced later):
119119
```
120120
Do not report regressions here, but rather in the corresponding issue: #ISSUE_NUMBER
121121
The changelog is also there.
122122
```
123-
- once you created the PR, note down its number (from now on called `PR_NUMBER`)
124-
- in case some issue should be closed when the release PR is merged, link them using the "Development" tab on the right, or add a "Fixes #...." in the PR description
125-
- *for example check out [#8231](https://github.com/TeamNewPipe/NewPipe/pull/8231) as a reference*
123+
- once you have created the PR, note down its number (from now on called `PR_NUMBER`)
124+
- in case some issue would be fixed when the release PR is merged, link them using the "Development" tab on the right, or add a "Fixes #...." in the PR description
125+
- *for example, check out [#8231](https://github.com/TeamNewPipe/NewPipe/pull/8231) for reference*
126126
127127
### Creating the issue
128128
@@ -135,17 +135,17 @@ Now there shuold be two new commits (the Weblate and changelog ones) on your loc
135135
Debug APK (built by our CI in #PR_NUMBER): ...
136136
Please report **only regressions** (i.e. new issues) here, not issues that were already present in the previous release!
137137
```
138-
- the `## TODO` section should contain a list of things that still need to be done before releasing, for example regressions that need to be fixed, or a reminder to merge the Weblate changelogs before releasing (use `- [ ]` to create checkbox lists)
138+
- an optional `## TODO` section should contain a list of things that still need to be done before releasing, for example regressions that need to be fixed, or a reminder to merge the Weblate changelogs before releasing (use `- [ ]` to create checkbox lists)
139139
- the `## NewPipeExtractor version` should contain a link to the NewPipeExtractor release this new NewPipe version will ship with (i.e. the one set in [Creating the release branch](#creating-the-release-branch))
140-
- copy-paste the draft Markdown changelog [kept on GitHub](https://github.com/TeamNewPipe/NewPipe/releases) (you finalized it earlier in [Create a changelog](#create-a-changelog)) and put it under the `## App changelog` section
141-
- once you created the issue, pin it using the "Pin issue" button on the right
142-
- *for example check out [#8230](https://github.com/TeamNewPipe/NewPipe/pull/8230) as a reference*
140+
- copy the draft Markdown changelog [kept on GitHub](https://github.com/TeamNewPipe/NewPipe/releases) (you finalized it earlier in [Create a changelog](#create-a-changelog)) to the clipboard and paste it under the `## App changelog` section
141+
- once you have created the issue, pin it using the "Pin issue" button on the right
142+
- *for example, check out [#8230](https://github.com/TeamNewPipe/NewPipe/issues/8230) for reference*
143143
144144
### Testing APKs
145145
146-
The first time you open the release issue and then each time some changes are made to the release PR, you should provide a debug APK in the `## Testing for regressions` section.
147-
- wait for the Continuous Integration (CI) to finish in the PR, then download from the "Checks" tab the debug APK it has built
148-
- rename it to `NewPipe_vX.X.X_RC1_debug.apk` where `RC1` should be incremented to `RC2`, ... each time a new APK is provided
146+
The first time you open the release issue, and then each time some changes are made to the release PR, you should provide a debug APK in the `## Testing for regressions` section.
147+
- wait for the Continuous Integration (CI) to finish testing the PR, then download the debug APK it will have built from the "Checks" tab
148+
- rename it to `NewPipe_vX.X.X_RC1_debug.apk` where `RC1` should be incremented to `RC2` and so on each time a new APK is provided
149149
- zip it and make sure the `.zip` file has the same name as the `.apk` it contains
150150
- upload it in the issue description, replacing the `...` placeholder used above
151151
@@ -179,7 +179,7 @@ Once enough time has passed and all regressions and TODOs have been solved, you
179179
- `git fetch weblate`
180180
- obtain the hash of the last commit on the `weblate/dev` remote branch
181181
- `git log -n 1 --pretty="format:%H" weblate/dev`
182-
- cherry pick the has you obtained above into the release branch (the one you are currently on)
182+
- cherry pick the hash you obtained above into the release branch (the one you are currently on)
183183
- `git cherry-pick HASH`
184184
- push the changes to the remote branch
185185
- `git push origin release-X.X.X`

0 commit comments

Comments
 (0)