Skip to content

Commit c48f6a5

Browse files
authored
Fix Discussions category name: cicd -> ci-cd (#207) (#224)
The first post-merge nightly failed the release-post-comment workflow with: Discussion category 'cicd' not found. Available: Announcements, ci-cd, General, Ideas, Polls, Q&A, Show and tell The repository's category is named "ci-cd", not "cicd". Update the default category name (workflow_dispatch input default, the env default, and the script fallback) and the changelog wording accordingly. The lookup remains case-insensitive by name (no hard-coded category id). Note: This work was completed with AI assistance (Claude Code).
1 parent 508bc01 commit c48f6a5

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

.github/workflows/release-post-comment.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ on:
1313
description: "GitHub Discussions category name to post into (case-insensitive)"
1414
required: false
1515
type: string
16-
default: "cicd"
16+
default: "ci-cd"
1717

1818
permissions:
1919
contents: read # Required for reading releases
@@ -149,14 +149,14 @@ jobs:
149149
- name: Post to GitHub Discussions
150150
uses: actions/github-script@v7
151151
env:
152-
DISCUSSION_CATEGORY: ${{ github.event.inputs.discussion_category || 'cicd' }}
152+
DISCUSSION_CATEGORY: ${{ github.event.inputs.discussion_category || 'ci-cd' }}
153153
with:
154154
script: |
155155
const fs = require('fs');
156156
const body = fs.readFileSync('discussion-post.md', 'utf8');
157157
const releaseName = process.env.RELEASE_NAME;
158158
const title = `Release ${releaseName}`;
159-
const wantCategory = (process.env.DISCUSSION_CATEGORY || 'cicd').toLowerCase();
159+
const wantCategory = (process.env.DISCUSSION_CATEGORY || 'ci-cd').toLowerCase();
160160
161161
// Resolve the repository node id and the target discussion category id
162162
// by (case-insensitive) name, so we never hard-code a repo-specific

docs/changelog.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ This document contains a reverse-chronological list of changes to txaio.
1818
**Other**
1919

2020
* Preserve verified artifact chain-of-custody in the release pipeline, aligning txaio with the rest of the WAMP group. The wheel, source distribution and documentation are now built **once** in the ``main`` workflow and uploaded with ``upload-artifact-verified``; the ``release`` workflow ``download-artifact-verified``\s those exact, checksum-verified artifacts (by ``main`` run id) and publishes them, instead of rebuilding from source. This guarantees that what is tested is what is shipped (`#207 <https://github.com/crossbario/txaio/issues/207>`_)
21-
* Auto-publish a GitHub Discussions announcement for nightly and stable releases: a new ``release-post-comment.yml`` workflow runs after the ``release`` workflow, finds the freshly-created GitHub Release and posts its notes into the Discussions ``cicd`` category. The category is resolved by name (case-insensitive), so no repo-specific category id is hard-coded. Requires GitHub Discussions enabled on the repository with a ``cicd`` category (`#207 <https://github.com/crossbario/txaio/issues/207>`_)
21+
* Auto-publish a GitHub Discussions announcement for nightly and stable releases: a new ``release-post-comment.yml`` workflow runs after the ``release`` workflow, finds the freshly-created GitHub Release and posts its notes into the Discussions ``ci-cd`` category. The category is resolved by name (case-insensitive), so no repo-specific category id is hard-coded. Requires GitHub Discussions enabled on the repository with a ``ci-cd`` category (`#207 <https://github.com/crossbario/txaio/issues/207>`_)
2222
* Align artifact handling with the rest of the WAMP group: CI now uploads build, coverage and documentation artifacts via the shared ``upload-artifact-verified`` action (SHA256 chain-of-custody), and the release workflow validates the exact PyPI payload — the universal ``py3-none-any`` wheel plus the source distribution — with the shared, fail-closed ``check-release-fileset`` action before publishing (`#212 <https://github.com/crossbario/txaio/issues/212>`_)
2323
* Remove three permanently-skipped legacy tests that exercised the ``@asyncio.coroutine`` decorator. That decorator was removed in Python 3.11 (txaio's minimum supported version), so the tests could no longer run on any supported interpreter and only emitted skips (`#152 <https://github.com/crossbario/txaio/issues/152>`_)
2424
* Bumped the shared ``wamp-ai`` and ``wamp-cicd`` Git submodules to match the rest of the WAMP project group (zlmdb / autobahn-python 26.6.1) for the coordinated release. The ``wamp-cicd`` bump picks up the GHSA-6658 shell-injection hardening in the shared ``identifiers.yml`` reusable workflow (untrusted GitHub event fields are now passed via ``env:`` as quoted data with a fail-closed branch-name allowlist) (`#218 <https://github.com/crossbario/txaio/issues/218>`_)

0 commit comments

Comments
 (0)