Skip to content
Snippets Groups Projects

Draft: Add a CI check for dead links

Closed Kez requested to merge dead-link-check into main

Closes #14

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
199 199
200 200 check_dead_links:
201 201 stage: test
202 image: lycheeverse/lychee
202 image: debian:bullseye-slim
203 203 allow_failure: true
204 dependencies:
205 - build
  • Comment on lines +204 to +205

    I think this might explain the issues with previous artifacts. You don't need to specify dependencies: because by default it will download all artifacts from the previous pipeline stage. You would want it if there we multiple jobs in the build stage producing different artifacts and you wanted to retrieve only those from one (or more) of the jobs as opposed to all.

    Edited by Jérôme Charaoui
  • Kez changed this line in version 3 of the diff

    changed this line in version 3 of the diff

  • Author Contributor

    removed the key and cleared the ci cache, and that seems to have fixed it. thanks!

  • Please register or sign in to reply
  • Kez added 1 commit

    added 1 commit

    Compare with previous version

  • Kez added 1 commit

    added 1 commit

    • f34a56f8 - Add a CI check for dead links

    Compare with previous version

  • Kez added 1 commit

    added 1 commit

    • ff82fb25 - Add a CI check for dead links

    Compare with previous version

  • Kez added 1 commit

    added 1 commit

    • 58d1ca26 - Add a CI check for dead links

    Compare with previous version

  • Kez added 1 commit

    added 1 commit

    • 5674263a - Add a CI check for dead links

    Compare with previous version

  • Kez added 1 commit

    added 1 commit

    • 1d85e5ec - Add a CI check for dead links

    Compare with previous version

  • Kez added 1 commit

    added 1 commit

    Compare with previous version

  • Kez added 1 commit

    added 1 commit

    Compare with previous version

  • Kez added 1 commit

    added 1 commit

    Compare with previous version

  • Kez added 1 commit

    added 1 commit

    Compare with previous version

  • Kez added 1 commit

    added 1 commit

    Compare with previous version

  • Kez added 1 commit

    added 1 commit

    Compare with previous version

  • Author Contributor

    alright, i think i finally tweaked this enough. @anarcat you can see an example ci log from this here. it's running lychee with these args --no-progress --require-https --exclude '.*?\.onion$' --exclude-all-private to require links use https (if available), exclude onion urls (lychee doesn't support them, would be a good PR), and excludes all private IP addresses.

    it looks good to me, and it's actually already found a few broken links on donate-static.

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading