ci: check for correct spellings and secure practices in actions by zimeg · Pull Request #484 · slackapi/slack-github-action · GitHub
Skip to content

ci: check for correct spellings and secure practices in actions - #484

Draft
zimeg wants to merge 7 commits into
mainfrom
zimeg-ci-healthscore-prerelease
Draft

zimeg wants to merge 7 commits into
mainfrom
zimeg-ci-healthscore-prerelease

Conversation

@zimeg

@zimeg zimeg commented Aug 2, 2025

Copy link
Copy Markdown
Member

Summary

This PR tests the prerelease of slackapi/slack-health-score@v0.2.0-rc.1 for spell checks and secure practices.

Requirements

@zimeg zimeg self-assigned this Aug 2, 2025
@zimeg zimeg added the github_actions Pull requests that update GitHub Actions code label Aug 2, 2025
@codecov

codecov Bot commented Aug 2, 2025 •

Copy link
Copy Markdown

@github-advanced-security

Copy link
Copy Markdown

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

@zimeg

zimeg commented Aug 2, 2025

Copy link
Copy Markdown
Member Author

📣 The zizmor check completes and typos isn't run with the healthscore itself now erroring with the following:

Run actions/github-script@v7
Error [ERR_REQUIRE_ASYNC_MODULE]: require() cannot be used on an ESM graph with top-level await. Use import() instead. To see where the top-level await comes from, use --experimental-print-required-tla.
    at ModuleJobSync.runSync (node:internal/modules/esm/module_job:384:13)
    at ModuleLoader.importSyncForRequire (node:internal/modules/esm/loader:323:47)
    at loadESMFromCJS (node:internal/modules/cjs/loader:1371:24)
    at Module._compile (node:internal/modules/cjs/loader:1511:5)
    at Module._extensions..js (node:internal/modules/cjs/loader:1572:16)
    at Module.load (node:internal/modules/cjs/loader:1275:32)
    at Module._load (node:internal/modules/cjs/loader:1096:12)
    at Module.require (node:internal/modules/cjs/loader:1298:19)
    at require (node:internal/modules/helpers:182:18)
    at Object.apply (/home/runner/work/_actions/actions/github-script/v7/dist/index.js:35467:27) {
  code: 'ERR_REQUIRE_ASYNC_MODULE'
}
Error: Unhandled error: Error [ERR_REQUIRE_ASYNC_MODULE]: require() cannot be used on an ESM graph with top-level await. Use import() instead. To see where the top-level await comes from, use --experimental-print-required-tla.

🔗 https://github.com/slackapi/slack-github-action/actions/runs/16688509618/job/47242358352?pr=484#step:31:208

@zimeg

zimeg commented Aug 2, 2025

Copy link
Copy Markdown
Member Author

📣 Now we run the checkups and attempt to read a missing .typos.toml from the health score:

$ ./typos . --config .typos.toml
could not read config at `.typos.toml`

🔗 https://github.com/slackapi/slack-github-action/actions/runs/16689275015/job/47244330326#step:31:4605

@zimeg

zimeg commented Aug 2, 2025

Copy link
Copy Markdown
Member Author

This branch had an error being deployed

1 failed deployment
staging — a958b2c5 Deployed Aug 2, 2025 by zimeg
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github_actions Pull requests that update GitHub Actions code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants