Skip to content

fix: replace CircleCI env vars in post-release Slack notifications#230

Merged
adekbadek merged 1 commit intotrunkfrom
fix/post-release-slack-messages
Apr 9, 2026
Merged

fix: replace CircleCI env vars in post-release Slack notifications#230
adekbadek merged 1 commit intotrunkfrom
fix/post-release-slack-messages

Conversation

@adekbadek
Copy link
Copy Markdown
Member

@adekbadek adekbadek commented Apr 1, 2026

Summary

  • Replaces $CIRCLE_PROJECT_REPONAME and $CIRCLE_BUILD_URL with GitHub Actions equivalents ($GITHUB_REPOSITORY, $GITHUB_SERVER_URL/…/runs/$GITHUB_RUN_ID) in post-release Slack notifications
  • Removes duplicate root-level scripts (post-release.sh, release-wporg.sh) – the scripts/github/ versions are the canonical ones

🤖 Generated with Claude Code

@adekbadek adekbadek force-pushed the fix/post-release-slack-messages branch from a0c3110 to a313947 Compare April 1, 2026 07:42
Also removes duplicate root-level scripts (post-release.sh,
release-wporg.sh) in favor of scripts/github/ versions, and updates
the legacy CircleCI job reference in src/jobs/post-release.yml.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@adekbadek adekbadek force-pushed the fix/post-release-slack-messages branch from a313947 to d6aca70 Compare April 1, 2026 07:44
@adekbadek adekbadek marked this pull request as ready for review April 1, 2026 07:45
@adekbadek adekbadek requested a review from a team as a code owner April 1, 2026 07:45
Comment thread src/jobs/post-release.yml
- run:
name: Perform post-release chores
command: ./node_modules/newspack-scripts/post-release.sh
command: ./node_modules/newspack-scripts/scripts/github/post-release.sh
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This yml file is for circleci and is not used in gh actions. Now that the migration is complete we could remove src/jobs entirely.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

That would break the CircleCI orb, I'd say this is out of scope and can be tackled in another PR – along with making sure nothing consumes the orb anymore.

@adekbadek adekbadek merged commit b4fa032 into trunk Apr 9, 2026
6 checks passed
@adekbadek adekbadek deleted the fix/post-release-slack-messages branch April 9, 2026 20:25
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 9, 2026

🎉 This PR is included in version 5.9.7 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants