Skip to content

chore: release config fixes #1014 - #1015

Open
mart-sessman wants to merge 2 commits into
rcfrom
fix/1014-release-success-comment
Open

mart-sessman wants to merge 2 commits into
rcfrom
fix/1014-release-success-comment

Conversation

@mart-sessman

@mart-sessman mart-sessman commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Closes #1014

Part of TEDI-Design-System/general#158

  • Moves the release config from .releaserc.json to release.config.mjs.
  • Disables success comments with successCommentCondition: false. A reference to #0 failed the release in this step, and the comments were never posted anyway (the release job has no permission to comment).
  • Links owner/repo#123 in commit subjects to the repo they name instead of this repo.

The success step looks up every referenced issue via GraphQL, so a reference to #0 fails the release before any condition is evaluated. The comments were never posted anyway: the release job has no issues or pull-requests write permission.
A JS config lets the release notes point owner/repo#123 references at the repo they name instead of this repo.
@coderabbitai

coderabbitai Bot commented Oct 20, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 55 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Repository: TEDI-Design-System/react/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 530c0f4f-eb6d-4b7b-9bb4-1684eda0cbd6
📥 Commits

Reviewing files that changed from the base of the PR and between 393df67 and e57ca51.

📒 Files selected for processing: 2
  • .releaserc.json
  • release.config.mjs
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codecov

codecov Bot commented Oct 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@github-actions

Copy link
Copy Markdown

♿ Accessibility — ✅ no blocking violations

No accessibility violations in the components changed by this PR.

⚠️ Pre-existing — 5 violations in components not changed here (warn only)

Component Rules Stories
TEDI-Ready/Components/Buttons/CollapseButton unknown ×2 2
TEDI-Ready/Components/Form/TextField unknown ×2 2
TEDI-Ready/Components/Navigation/Tabs unknown 1
Stories

@github-actions

Copy link
Copy Markdown

🎨 Chromatic

116 visual change(s) to review before merging.

Built from e57ca51. Commits pushed after this are not covered; approve again to rebuild.

View the build

This branch was successfully deployed

1 active deployment
github-pages — e57ca514 Deployed Oct 20, 2026 by mart-sessman via Deploy #1859
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Release config: fails on a reference to #0, cross-repo issue links point to the wrong repo

2 participants