Skip to content

Use args with status updater instead of env vars, update checkout to v3#133

Merged
Nuru merged 2 commits intomasterfrom
status-updater
Sep 30, 2022
Merged

Use args with status updater instead of env vars, update checkout to v3#133
Nuru merged 2 commits intomasterfrom
status-updater

Conversation

@Nuru
Copy link
Contributor

@Nuru Nuru commented Sep 30, 2022

what

  • Use command line arguments to github-status-updater instead of environment variables
  • Update actions/checkout from v2 to current v3

why

  • The environment variables github-status-updater uses conflict with the GitHub Actions reserved namespace. In particular, sometime around 2022-09-28, GitHub actions began setting GITHUB_STATE, overriding the input to github-status-updater and causing it to fail. This in turn cause all our automated testing to fail.
  • Security updates, use more current version of NodeJS

references

@Nuru Nuru requested review from aknysh and osterman September 30, 2022 22:51
@Nuru Nuru added bugfix Change that restores intended behavior patch A minor, backward compatible change labels Sep 30, 2022
@Nuru Nuru merged commit b6da961 into master Sep 30, 2022
@Nuru Nuru deleted the status-updater branch September 30, 2022 23:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Change that restores intended behavior patch A minor, backward compatible change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant