From c8e308fa674234296318567f46d317db871d640e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Lucas=20Serv=C3=A9n=20Mar=C3=ADn?= Date: Tue, 19 Apr 2022 13:42:15 +0200 Subject: [PATCH] docs: fix URLs preventing generation and unblock CI (#5285) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * docs: fix Ian Billett's GitHub handle I noticed that CI was failing [0] for PR https://github.com/thanos-io/thanos/pull/5284 because Ian had changed his GitHub handle from @ianbillett to @bill3tt. This commit fixes this. [0] https://github.com/thanos-io/thanos/runs/6050355497?check_suite_focus=true#step:5:135 Signed-off-by: Lucas Servén Marín * docs: fix broken links to GitHub docs Currently, documentation generation is failing because mdox can't fetch some GitHub documentation pages since the URLs for the help content has changed. This commit updates the links to use the correct URLs. Signed-off-by: Lucas Servén Marín * MAINTAINERS.md: regenerate Signed-off-by: Lucas Servén Marín --- .mdox.validate.yaml | 3 +++ MAINTAINERS.md | 20 +++++++++---------- docs/release-process.md | 6 +++--- tutorials/katacoda/thanos/3-receiver/intro.md | 2 +- 4 files changed, 17 insertions(+), 14 deletions(-) diff --git a/.mdox.validate.yaml b/.mdox.validate.yaml index 0564d5cb75..15afd4eab7 100644 --- a/.mdox.validate.yaml +++ b/.mdox.validate.yaml @@ -20,3 +20,6 @@ validators: # couldn't reach even when curl-ed. - regex: 'cloud\.baidu\.com' type: 'ignore' + # 403 when curl-ed from GitHub actions, though not from a developer machine. Likely due to secondary rate limits. + - regex: 'docs\.github\.com' + type: 'ignore' diff --git a/MAINTAINERS.md b/MAINTAINERS.md index 44bfb58bd0..236b7e10a8 100644 --- a/MAINTAINERS.md +++ b/MAINTAINERS.md @@ -24,19 +24,19 @@ You can reach us under `thanos-io@googlegroups.com` email. ## Triage -We also have some nice souls that help triaging issues and PRs. See [here](https://help.github.com/en/articles/repository-permission-levels-for-an-organization#permission-levels-for-repositories-owned-by-an-organization) for details about the role's permission. +We also have some nice souls that help triaging issues and PRs. See [here](https://docs.github.com/en/organizations/managing-access-to-your-organizations-repositories/repository-roles-for-an-organization#repository-roles-for-organizations) for details about the role's permission. Full list of triage persons is displayed below: -| Name | Slack | GitHub | Company | -|---------------|---------------|----------------------------------------------|---------| -| Adrien Fillon | `@Adrien F` | [@adrien-f](https://github.com/adrien-f) | | -| Ian Billett | `@billett` | [@ianbillett](https://github.com/ianbillett) | Red Hat | -| Martin Chodur | `@FUSAKLA` | [@fusakla](https://github.com/fusakla) | | -| Michael Dai | `@jojohappy` | [@jojohappy](https://github.com/jojohappy) | | -| Xiang Dai | `@daixiang0` | [@daixiang0](https://github.com/daixiang0) | | -| Jimmie Han | `@hanjm` | [@hanjm](https://github.com/hanjm) | Tencent | -| Matej Gera | `@Matej Gera` | [@matej-g](https://github.com/matej-g) | Red Hat | +| Name | Slack | GitHub | Company | +|---------------|---------------|--------------------------------------------|---------| +| Adrien Fillon | `@Adrien F` | [@adrien-f](https://github.com/adrien-f) | | +| Ian Billett | `@billett` | [@bill3tt](https://github.com/bill3tt) | Red Hat | +| Martin Chodur | `@FUSAKLA` | [@fusakla](https://github.com/fusakla) | | +| Michael Dai | `@jojohappy` | [@jojohappy](https://github.com/jojohappy) | | +| Xiang Dai | `@daixiang0` | [@daixiang0](https://github.com/daixiang0) | | +| Jimmie Han | `@hanjm` | [@hanjm](https://github.com/hanjm) | Tencent | +| Matej Gera | `@Matej Gera` | [@matej-g](https://github.com/matej-g) | Red Hat | Please reach any of the maintainer on slack or email if you want to help as well. diff --git a/docs/release-process.md b/docs/release-process.md index fad32b31df..d80f19d6cf 100644 --- a/docs/release-process.md +++ b/docs/release-process.md @@ -117,11 +117,11 @@ Feel free to mimic following PR: https://github.com/thanos-io/thanos/pull/3861 git push origin "v${tag}" ``` - Signing a tag with a GPG key is appreciated, but in case you can't add a GPG key to your Github account using the following [procedure](https://help.github.com/articles/generating-a-gpg-key/), you can replace the `-s` flag by `-a` flag of the `git tag` command to only annotate the tag without signing. + Signing a tag with a GPG key is appreciated, but in case you can't add a GPG key to your GitHub account using the following [procedure](https://docs.github.com/en/authentication/managing-commit-signature-verification/generating-a-new-gpg-key), you can replace the `-s` flag by `-a` flag of the `git tag` command to only annotate the tag without signing. Please make sure that you are tagging the merge commit because otherwise GitHub's UI will show that there were more commits after your release. -2. Once a tag is created and pushed, **immediately** create a Github Release using the UI for this tag, as otherwise CircleCI will not be able to upload tarballs for this tag. Go to the releases page of the project, click on the `Draft a new release` button and select the tag you just pushed. Describe release and post relevant entry from changelog. Click `Save draft` **rather** than `Publish release` at this time. (This will prevent the release being visible before it has got the binaries attached to it.) *In case you did not manage to create the draft release before CircleCI run is finished (it will fail on the artifacts upload step in this case), you can re-trigger the run manually from the CircleCI dashboard *after* you created the draft release.* +2. Once a tag is created and pushed, **immediately** create a GitHub Release using the UI for this tag, as otherwise CircleCI will not be able to upload tarballs for this tag. Go to the releases page of the project, click on the `Draft a new release` button and select the tag you just pushed. Describe release and post relevant entry from changelog. Click `Save draft` **rather** than `Publish release` at this time. (This will prevent the release being visible before it has got the binaries attached to it.) *In case you did not manage to create the draft release before CircleCI run is finished (it will fail on the artifacts upload step in this case), you can re-trigger the run manually from the CircleCI dashboard *after* you created the draft release.* 3. You are also encouraged to include a list of (first time) contributors to the release. You can do this by clicking on `Auto-generate release notes`, which will generate this section for you (edit the notes as required to remove unnecessary parts). @@ -149,5 +149,5 @@ Feel free to mimic following PR: https://github.com/thanos-io/thanos/pull/3861 The following changes to the above procedures apply: * In line with [Semantic Versioning](http://semver.org/), append something like `-rc.0` to the version (with the corresponding changes to the tag name, the release name etc.). -* Tick the `This is a pre-release` box when drafting the release in the Github UI. +* Tick the `This is a pre-release` box when drafting the release in the GitHub UI. * Still update `CHANGELOG.md`, but when you cut the final release later, merge all the changes from the pre-releases into the one final update. diff --git a/tutorials/katacoda/thanos/3-receiver/intro.md b/tutorials/katacoda/thanos/3-receiver/intro.md index 428dbd7218..81cdc76fac 100644 --- a/tutorials/katacoda/thanos/3-receiver/intro.md +++ b/tutorials/katacoda/thanos/3-receiver/intro.md @@ -23,4 +23,4 @@ Let us know on https://github.com/thanos-io/thanos or #thanos slack channel link ### Contributed by: -* Ian Billett [@ianbillett](http://github.com/ianbillett) \ No newline at end of file +* Ian Billett [@bill3tt](http://github.com/bill3tt)