Skip to content

Conversation

@SbloodyS
Copy link
Member

@SbloodyS SbloodyS commented Sep 2, 2025

Purpose of the pull request

fix error in https://github.com/apache/dolphinscheduler/actions/runs/17365143434/job/49300041965?pr=17461

replace markdown-link-check with lychee to fix errors and improve performance.

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.

(or)

This pull request is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

(or)

Pull Request Notice

Pull Request Notice

If your pull request contains incompatible change, you should also add it to docs/docs/en/guide/upgrade/incompatible.md

@SbloodyS SbloodyS requested a review from EricGao888 as a code owner September 2, 2025 02:11
@SbloodyS SbloodyS self-assigned this Sep 2, 2025
@SbloodyS SbloodyS added this to the 3.3.2 milestone Sep 2, 2025
markdown-link-check -c .dlc.json -q "$file" &
done
wait || { EXIT_CODE=$?; cleanup; }
lychee -c .github/workflows/lychee.toml docs/docs/*
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add -v to print more message?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 2, 2025

@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 2, 2025

Please retry analysis of this Pull-Request directly on SonarQube Cloud

@SbloodyS SbloodyS merged commit a9b829d into apache:dev Sep 2, 2025
90 of 141 checks passed
@SbloodyS SbloodyS deleted the chore_fix_doc_deadlink_check branch September 2, 2025 09:06
davidzollo pushed a commit to davidzollo/dolphinscheduler that referenced this pull request Oct 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants