From a718c89e0791828326a2d3980b30d45ce7fd2de1 Mon Sep 17 00:00:00 2001 From: XiangRongLin <41164160+XiangRongLin@users.noreply.github.com> Date: Sun, 28 Nov 2021 21:01:59 +0100 Subject: [PATCH] Fix most errors from linkcheck --- docs/00_Prepare_everything.md | 2 +- docs/03_Implement_a_service.md | 2 +- docs/06_releasing.md | 10 +++++----- linkcheck.py | 2 +- 4 files changed, 8 insertions(+), 8 deletions(-) diff --git a/docs/00_Prepare_everything.md b/docs/00_Prepare_everything.md index 39bca47..a1b6315 100644 --- a/docs/00_Prepare_everything.md +++ b/docs/00_Prepare_everything.md @@ -25,7 +25,7 @@ First and foremost, you need to meet the following conditions in order to write - A dev environment/IDE that supports: - __[Git](https://git-scm.com/downloads/guis)__ - - __[Java 8](https://www.java.com/en/download/faq/java8.xml)__ + - __[Java 8](https://adoptium.net/?variant=openjdk8&jvmVariant=hotspot)__ - __[Gradle](https://gradle.org/)__ - __[Unit testing](https://junit.org/junit5/)__ - [IDEA Community](https://www.jetbrains.com/idea/) (Strongly recommended, but not required) diff --git a/docs/03_Implement_a_service.md b/docs/03_Implement_a_service.md index 120cf49..515c4a2 100644 --- a/docs/03_Implement_a_service.md +++ b/docs/03_Implement_a_service.md @@ -34,7 +34,7 @@ ask us to introduce a new library than start using Regex too often. - Html/XML Parsing: [jsoup](https://jsoup.org/apidocs/overview-summary.html) - JSON Parsing: [nanojson](https://github.com/mmastrac/nanojson#parser-example) -- JavaScript Parsing/Execution: [Mozilla Rhino](https://developer.mozilla.org/en-US/docs/Mozilla/Projects/Rhino/Documentation) +- JavaScript Parsing/Execution: [Mozilla Rhino](https://github.com/mozilla/rhino) - Link detection in strings: [AutoLink](https://github.com/robinst/autolink-java) If you need to introduce new libraries, please tell us before you do so. diff --git a/docs/06_releasing.md b/docs/06_releasing.md index 1a0ea1a..0256f56 100644 --- a/docs/06_releasing.md +++ b/docs/06_releasing.md @@ -74,9 +74,9 @@ For NewPipe, this means: __Don't do a release if you don't have time for it!!!__ By following the steps listed below, you can publish a stable version of NewPipe: -1. [Merge the translations from Weblate into NewPipe](../07_maintainers_view#merge-changes-from-weblate-into-newpipe). +1. [Merge the translations from Weblate into NewPipe](../08_maintainers_view#merge-changes-from-weblate-into-newpipe). 2. Update your local __dev__ branch and create a [changelog file](#changelog-file). - Make sure to push the changes and [update Weblate](../07_maintainers_view#update-weblate). + Make sure to push the changes and [update Weblate](../08_maintainers_view#update-weblate). 3. In your local NewPipe repo, fork the __dev__ branch into a new __release/x.y.z__ branch. 4. Increase the [version number](#version-nomenclature) and update the version code in the `build.gradle` file. 5. Open a pull request form the new __release/x.y.z__ branch into the __master__ branch. @@ -96,7 +96,7 @@ PRs which aim to fix regressions of the upcoming release need to target the __re Read [Quickfixes](#quickfixes) for more info. The changelogs are translated during the test phase. -Therefore, [the translations need to be merged from Weblate](../07_maintainers_view#merge-changes-from-weblate-into-newpipe) once more. +Therefore, [the translations need to be merged from Weblate](../08_maintainers_view#merge-changes-from-weblate-into-newpipe) once more. The translation commit is cherry-picked into the release branch. Once testing is done and the release branch does not contain critical regressions, and you think the update is ready, @@ -122,7 +122,7 @@ _Don't update the actual version number. :P_ ### Releasing Once the glorious day of all days has come, and you fulfill the ceremony of releasing. -After going through the release procedure of [creating a new release](#create_a_new_release) +After going through the release procedure of [creating a new release](#creating-a-new-release) and maybe a few [quickfixes](#quickfixes) on the new release, this is what you should do when releasing: @@ -275,7 +275,7 @@ For this reason it is recommended to keep the changelog at 400 bytes. When creating the changelog file be aware of changes which were done in the extractor as well. Before pushing the changelog to NewPipe's repo, ask other maintainers to review it. -After pushing the changelog to NewPipe's GitHub repo, [updating Weblate](07_maintainers_view/#update-webalte) is necessary. +After pushing the changelog to NewPipe's GitHub repo, [updating Weblate](../08_maintainers_view#update-weblate) is necessary. This enables translators to work on localized versions of the changelog before a release is tagged and published. ## Publish on F-Droid diff --git a/linkcheck.py b/linkcheck.py index 63f358c..608dff4 100755 --- a/linkcheck.py +++ b/linkcheck.py @@ -18,7 +18,7 @@ for filename in os.listdir(): for link in re.findall('\[.*\]\(([^\)]*)\)', filedata): if link.startswith("#"): checkstring = "# " + link.replace("#", "").replace("-", " ") - if not checkstring in filedata.lower(): + if not checkstring in filedata.lower().replace("-", " "): RETURN_VALUE = 1 print(filename + ": Could not find target for " + link) else: