Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
48 changes: 25 additions & 23 deletions .github/workflows/check-links.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,46 +2,48 @@ name: check-links
on:
push:
branches: [ main ]
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: Should we add paths to on push as well?

Copy link
Author

Choose a reason for hiding this comment

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

I'll add paths to push as well to keep it consistent.

paths:
- '**/*.md'
- '.github/workflows/check-links.yml'
- '.github/workflows/check_links_config.json'
pull_request:
paths:
- '**/*.md'
- '.github/workflows/check-links.yml'
- '.github/workflows/check_links_config.json'

permissions:
contents: read

concurrency:
group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.run_id }}
cancel-in-progress: true

jobs:
changedfiles:
name: changed files
runs-on: ubuntu-latest
if: ${{ github.actor != 'dependabot[bot]' }}
outputs:
md: ${{ steps.changes.outputs.md }}
steps:
- name: Checkout Repo
uses: actions/checkout@v4
with:
fetch-depth: 0
- name: Get changed files
id: changes
run: |
echo "md=$(git diff --name-only --diff-filter=ACMRTUXB $(git merge-base origin/main ${{ github.event.pull_request.head.sha }}) ${{ github.event.pull_request.head.sha }} | grep .md$ | xargs)" >> $GITHUB_OUTPUT
check-links:
runs-on: ubuntu-latest
needs: changedfiles
if: |
github.event.pull_request.user.login != 'otelbot[bot]' && github.event_name == 'pull_request'
&& ${{needs.changedfiles.outputs.md}}
if: ${{ github.actor != 'dependabot[bot]' && github.actor != 'otelbot[bot]' }}
timeout-minutes: 15
steps:
- name: Checkout Repo
uses: actions/checkout@v4
uses: actions/checkout@v6

- name: Get changed markdown files
id: changed-files
uses: tj-actions/changed-files@v46
with:
fetch-depth: 0
files: |
**/*.md

- name: Install markdown-link-check
if: steps.changed-files.outputs.any_changed == 'true'
run: npm install -g markdown-link-check@v3.12.2

- name: Run markdown-link-check
if: steps.changed-files.outputs.any_changed == 'true'
run: |
markdown-link-check \
--verbose \
--config .github/workflows/check_links_config.json \
${{needs.changedfiles.outputs.md}} \
|| { echo "Check that anchor links are lowercase"; exit 1; }
${{ steps.changed-files.outputs.all_changed_files }} \
|| { echo "Check that anchor links are lowercase"; exit 1; }
3 changes: 2 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
([#4935](https://github.com/open-telemetry/opentelemetry-python/pull/4935))
- `opentelemetry-sdk`: upgrade vendored OTel configuration schema from v1.0.0-rc.3 to v1.0.0
([#4965](https://github.com/open-telemetry/opentelemetry-python/pull/4965))

- improve check-links ci job
([#4978](https://github.com/open-telemetry/opentelemetry-python/pull/4978))
## Version 1.40.0/0.61b0 (2026-03-04)

- `opentelemetry-sdk`: deprecate `LoggingHandler` in favor of `opentelemetry-instrumentation-logging`, see `opentelemetry-instrumentation-logging` documentation
Expand Down