Repo: Add CI job testing that yarn generate:configs still works #5925
Labels
accepting prs
Go ahead, send a pull request that resolves this issue
repo maintenance
things to do with maintenance of the repo, and not with code/docs
Suggestion
#5251 (comment):
yarn generate:configs
doesn't actually... work right now locally.#4952 bumped
chalk
to v5, which is an ESM-only version. You can read about the context about ESM-only packages here: https://gist.github.com/sindresorhus/a39789f98801d908bbc7ff3ecc99d99c.Unfortunately we're not yet on pure ESM for our source scripts, and so cannot use this new version of
chalk
. We'll have to downgradechalk
to v4.We don't have any CI task verifying that
yarn generate:configs
does not crash - so we have no way of knowing that any change breaks that particular script, unless we think to explicitly test it.Proposal: let's add a new job to
ci.yml
that verifiesyarn generate:configs
does not exit with an error code?The text was updated successfully, but these errors were encountered: