Skip to content

Commit 1916be7

Browse files
authored
Merge pull request #11993 from Calinou/ci-fix-lychee
Fix Lychee URL checking on CI
2 parents b226afc + fcadbdf commit 1916be7

8 files changed

Lines changed: 24 additions & 11 deletions

File tree

.github/workflows/check_urls.yml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,13 +24,14 @@ jobs:
2424
uses: lycheeverse/lychee-action@v2
2525
with:
2626
args: >
27-
--base .
27+
--root-dir .
2828
--no-progress
2929
--cache
3030
--max-cache-age 1d
31+
--exclude-path _build/
3132
--exclude-path _templates/
3233
--exclude-path classes/
33-
"**/*.md" "**/*.html" "**/*.rst"
34+
"**/*.md" "**/*.rst"
3435
3536
- name: Fail if there were link errors
3637
run: exit ${{ steps.lc.outputs.exit_code }}

.gitignore

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -61,3 +61,6 @@ tmp-unused-images-history
6161

6262
# Jetbrains IDE files
6363
/.idea/
64+
65+
# Lychee URL checking cache
66+
.lycheecache

.lycheeignore

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,18 +10,27 @@ tcp:\/\/.*
1010
# Security checks prevent checking the URLs of these websites automatically,
1111
# typically returning 403 errors.
1212
.*asecuritysite\.com.*
13+
.*computerhope\.com*
14+
.*developer\.arm\.com.*
15+
.*guru3d\.com.*
16+
.*inkscape\.org.*
1317
.*intel\.com.*
18+
.*itch\.io.*
19+
.*khronos\.org.*
20+
.*medium\.com.*
1421
.*reddit\.com.*
15-
.*inkscape.org.*
16-
.*computerhope.com*
22+
.*shadertoy\.com.*
1723

1824
# Don't check URLs from these websites due to frequent rate limits (error 429) or timeouts.
1925
.*adobe\.com.*
26+
.*developer\.apple\.com.*
27+
.*flathub\.org.*
2028
.*gamedevartisan\.com.*
2129
.*github\.com.*
2230
.*gnu\.org.*
2331
.*loopit\.dk.*
2432
.*meta\.com.*
33+
.*nightly\.link.*
2534
.*sourceforge\.io.*
2635

2736
# Not a valid URL with the GET method, which lychee always sends.

engine_details/architecture/core_types.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,7 @@ scripting API.
117117
| | | otherwise. |
118118
+-----------------------+--------------------------+---------------------------------------------------------------------------------------+
119119
| |rb_map| | ``std::map`` | Map type that uses a |
120-
| | | `red-black tree <https://en.wikipedia.org/wiki/Red-black-tree>`__ to find keys. |
120+
| | | `red-black tree <https://en.wikipedia.org/wiki/Red-black_tree>`__ to find keys. |
121121
| | | The performance benefits of ``RBMap`` aren't established, so prefer using other types.|
122122
+-----------------------+--------------------------+---------------------------------------------------------------------------------------+
123123
| |dictionary| 📜 | ``std::unordered_map`` | Keys and values can be of any Variant type. No static typing is imposed. |

engine_details/engine_api/custom_modules_in_cpp.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -466,7 +466,7 @@ Now if you go to the ``godot/modules/summator/doc_classes`` folder, you will see
466466
that it contains a ``Summator.xml`` file, or any other classes, that you referenced
467467
in your ``get_doc_classes`` function.
468468

469-
Edit the file(s) following the `class reference primer <https://contributing.godotengine.org/en/latest/documentation/class_reference/class_reference_primer.html>`__ and recompile the engine.
469+
Edit the file(s) following the `class reference primer <https://docs.godotengine.org/en/latest/engine_details/class_reference/index.html>`__ and recompile the engine.
470470

471471
Once the compilation process is finished, the docs will become accessible within
472472
the engine's built-in documentation system.

tutorials/animation/creating_movies.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -482,7 +482,7 @@ only 5.2 seconds of video after that point:
482482
ffmpeg -i input.avi -ss 00:00:12.10 -t 00:00:05.20 -crf 15 output.mp4
483483

484484
Cutting videos can also be done with the GUI tool
485-
`LosslessCut <https://mifi.github.io/lossless-cut/>`__.
485+
`LosslessCut <https://losslesscut.app/>`__.
486486

487487
Resizing video
488488
~~~~~~~~~~~~~~

tutorials/animation/playing_videos.rst

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -208,9 +208,9 @@ below with almost any input video format (AVI, MOV, WebM, …).
208208
.. this warning can be removed. That will likely be FFmpeg 7.2 or 8.0, and will
209209
.. likely happen during the Godot 4.5 or 4.6 release cycle.
210210
.. Commits fixing the issues:
211-
.. - https://github.org/FFmpeg/FFmpeg@22aa71d4da37a4ad2b0d28deeace64b57aa2ef50
212-
.. - https://github.org/FFmpeg/FFmpeg@84d85e7ad4ace228265af0c8c5caccc0730042fd
213-
.. - https://github.org/FFmpeg/FFmpeg@6e26f57f672b05e7b8b052007a83aef99dc81ccb
211+
.. - https://github.com/FFmpeg/FFmpeg/commit/22aa71d4da37a4ad2b0d28deeace64b57aa2ef50
212+
.. - https://github.com/FFmpeg/FFmpeg/commit/84d85e7ad4ace228265af0c8c5caccc0730042fd
213+
.. - https://github.com/FFmpeg/FFmpeg/commit/6e26f57f672b05e7b8b052007a83aef99dc81ccb
214214
215215
Balancing quality and file size
216216
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

tutorials/export/exporting_for_android.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ Launcher icons are used by Android launcher apps to represent your application t
103103
There are three types of icons:
104104

105105
- **Main Icon:** The "classic" icon. This will be used on all Android versions up to Android 8 (Oreo), exclusive. Must be at least 192×192 px.
106-
- **Adaptive Icons:** Starting from Android 8 (inclusive), `Adaptive Icons <https://developer.android.com/guide/practices/ui_guidelines/icon_design_adaptive>`_ were introduced. Applications will need to include separate background and foreground icons to have a native look. The user's launcher application will control the icon's animation and masking. Must be at least 432×432 px.
106+
- **Adaptive Icons:** Starting from Android 8 (inclusive), `Adaptive Icons <https://developer.android.com/develop/ui/compose/system/icon_design_adaptive>`_ were introduced. Applications will need to include separate background and foreground icons to have a native look. The user's launcher application will control the icon's animation and masking. Must be at least 432×432 px.
107107
- **Themed Icons (optional):** Starting from Android 13 (inclusive), Themed Icons were introduced. Applications will need to include a monochrome icon to enable this feature. The user's launcher application will control the icon's theme. Must be at least 432×432 px.
108108

109109
.. seealso:: It's important to adhere to some rules when designing adaptive icons. `Google Design has provided a nice article <https://medium.com/google-design/designing-adaptive-icons-515af294c783>`_ that helps to understand those rules and some of the capabilities of adaptive icons.

0 commit comments

Comments
 (0)