Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: Fix issue response script #24891

Merged
merged 15 commits into from
Feb 14, 2025
Merged

ci: Fix issue response script #24891

merged 15 commits into from
Feb 14, 2025

Conversation

maxdeviant
Copy link
Member

@maxdeviant maxdeviant commented Feb 14, 2025

This PR fixes the issue response script.

There were a number of things preventing it from working:

  • The directory name used in the GitHub Action did not match the one on disk.
    • The script has been moved accordingly
  • ts-node does not support ESM.
    • ts-node seems unmaintained, so I changed the script to be plain JS that is type-checked with TypeScript.
  • The data being sent to the Slack API was invalid:
    • Each section block can only have a maximum of 3000 characters in the text field, so we need to break up the issue list across multiple sections.
    • We needed to escape &, <, and > characters in the issue titles.

Release Notes:

  • N/A

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Feb 14, 2025
@maxdeviant maxdeviant self-assigned this Feb 14, 2025
@maxdeviant maxdeviant enabled auto-merge (squash) February 14, 2025 19:25
@maxdeviant maxdeviant merged commit fc85ca0 into main Feb 14, 2025
11 checks passed
@maxdeviant maxdeviant deleted the push-qlzzupkolltl branch February 14, 2025 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed The user has signed the Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant