.github/workflows/changelog-check.yml (2)

47-54: Solid fix for API limitations using local git diff.

The switch to git diff with commit SHAs bypasses GitHub's API constraints effectively. The pattern ^website/blog/.*\.(md|mdx)$ correctly captures blog files, and the explanatory comments (lines 50–52) are helpful for future maintainers.

One minor note: the grep -v 'tags.yml' is redundant since the preceding grep already filters for .md|.mdx extensions. You can drop it for clarity, but it's harmless as-is.

- NEW_BLOG_FILES=$(git diff --name-only "$BASE_REF" "$HEAD_REF" | grep -E '^website/blog/.*\.(md|mdx)$' | grep -v 'tags.yml' || true)
+ NEW_BLOG_FILES=$(git diff --name-only "$BASE_REF" "$HEAD_REF" | grep -E '^website/blog/.*\.(md|mdx)$' || true)

---

101-105: Consistent application of git diff approach in success message step.

The same pattern is correctly applied here. However, BASE_REF and HEAD_REF are extracted twice (lines 47–48 and 101–102). Consider moving these to a common step to reduce duplication and make future updates easier.

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

• Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

Learnt from: Listener430
Repo: cloudposse/atmos PR: 934
File: tests/fixtures/scenarios/docs-generate/README.md.gotmpl:99-118
Timestamp: 2025-01-25T03:51:57.689Z
Learning: For the cloudposse/atmos repository, changes to template contents should be handled in dedicated PRs and are typically considered out of scope for PRs focused on other objectives.

Learnt from: aknysh
Repo: cloudposse/atmos PR: 944
File: go.mod:206-206
Timestamp: 2025-01-17T00:18:57.769Z
Learning: For indirect dependencies with license compliance issues in the cloudposse/atmos repository, the team prefers to handle them in follow-up PRs rather than blocking the current changes, as these issues often require deeper investigation of the dependency tree.