Skip to content

Changelog validator: ignore markdown & configs d46223 #83618

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
simison merged 2 commits into from
Sep 28, 2026
Merged

Changelog validator: ignore markdown & configs d46223 #83618

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
f0cc0e2
CI: Ignore Markdown and Storybook files in CHANGELOG presence check
simison Sep 28, 2026
717f802
CI: Ignore root dev tooling configs in CHANGELOG presence check
simison Sep 28, 2026
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
Next Next commit
CI: Ignore Markdown and Storybook files in CHANGELOG presence check
  • Loading branch information
@simison
simison committed Sep 28, 2026
commit f0cc0e24df51edb1a6e80aa367546e8f36e9bc29
9 changes: 8 additions & 1 deletion .github/workflows/check-package-changelogs.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -41,12 +41,17 @@ on:
- 'packages/widget-dashboard/**'
- 'packages/widget-primitives/**'
- 'packages/wp-build/**'
- '!packages/*/**/*.md'
- 'packages/*/CHANGELOG.md'
- 'tools/validation/validate-changelogs.ts'
- '.github/workflows/check-package-changelogs.yml'
- '!packages/*/**/stories/**'
- '!packages/*/**/test/**'
- '!packages/*/**/__tests__/**'
- '!packages/*/**/*.stories.*'
- '!packages/*/**/*.mdx'
- '!packages/*/**/tsconfig.stories.json'
- '!packages/*/**/storybook.d.ts'

# Disable permissions for all available scopes by default.
# Any needed permissions should be configured at the job level.
Expand Down Expand Up @@ -127,8 +132,10 @@ jobs:
# The workflow's `paths:` filter triggers on the union of all
# tracked packages, so this matrix entry may run on a PR that
# doesn't touch its package. Short-circuit if that's the case.
# Tests, Storybook files and Markdown files (docs, READMEs,
# the CHANGELOG itself) don't warrant a CHANGELOG entry.
relevant_changes=$(git diff --name-only "$BASE_SHA"...HEAD -- "${PACKAGE_PATH}/" \
| grep -Ev "^${PACKAGE_PATH}/((.+/)?(stories|test|__tests__)/|CHANGELOG\.md$)" || true)
| grep -Ev "^${PACKAGE_PATH}/((.+/)?(stories|test|__tests__)/|.+\.(md|mdx)$|(.+/)?([^/]+\.stories\.[^/]+|tsconfig\.stories\.json|storybook\.d\.ts)$)" || true)

if [ -z "$relevant_changes" ]; then
echo "No relevant changes under ${PACKAGE_PATH}; skipping CHANGELOG presence check."
Expand Down