Skip to content

Added GitHub Actions Slack notification on the LTS line.#1370

Open
martnpaneq wants to merge 5 commits into
master-v54from
i/4450-gh-actions-slack-notification-master-v54
Open

Added GitHub Actions Slack notification on the LTS line.#1370
martnpaneq wants to merge 5 commits into
master-v54from
i/4450-gh-actions-slack-notification-master-v54

Conversation

@martnpaneq
Copy link
Copy Markdown
Contributor

@martnpaneq martnpaneq commented May 22, 2026

🚀 Summary

Added ckeditor5-dev-ci-notify-github-actions-status, a GitHub Actions equivalent of ckeditor5-dev-ci-notify-circle-status that posts a Slack notification summarizing a failed workflow run (commit author, branch, run URL, duration). Intended to be invoked from a step or job gated by if: failure().


📌 Related issues

  • See ckeditor/ckeditor5-internal#4450

💡 Additional information

Optional: Notes on decisions, edge cases, or anything helpful for reviewers.

@martnpaneq martnpaneq changed the title I/4450 gh actions slack notification master v54 Added GitHub Actions Slack notification on the LTS line. May 22, 2026
Comment thread packages/ckeditor5-dev-ci/lib/format-message.js
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit e2b60bd. Configure here.

* @param {boolean} options.shouldHideAuthor
*/
export default async function formatMessage( options ) {
const commitDetails = await getCommitDetails( options.triggeringCommitUrl, options.githubToken );
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Enterprise repo links wrong host

Medium Severity

The Slack attachment still builds the Repository (branch) links from a hardcoded https://github.com base, while this change routes commit URLs, issue references, and API calls through GITHUB_SERVER_URL / apiUrl for GitHub Enterprise. On Enterprise, those repository and branch links point at the wrong host even when the rest of the message is correct.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit e2b60bd. Configure here.

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.

1 participant