Ben Drucker

Any reason why this would be needed over just monkey patching `flush`?

This is definitively fixed in `@actions/[email protected]` as described in the [blog post](https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/) ```sh npm info @actions/core ``` ``` @actions/[email protected] | MIT | deps: 2 | versions: 25 Actions core lib...

I came across this with a related need for context propagation that I can describe in further detail if needed: tracing. If you're using httprate-redis and wrap your Redis client...

It would be good to add an input for this (`checksum`) to compliment `version`. Verifying a binary downloaded from GitHub releases against a checksum also downloaded from the same release...

> I think it's a good idea to input a checksum, but it can be a bit annoying for a user. Yep, mostly an option for users who want a...

> cache Doable, PR welcome > token Not sure about this. This is not the same situation as `setup-terraform`, which can accept Terraform credentials and write them somewhere where they...

> I'd only be inclined to do something like this by default with an alternative env var Clarifying: it probably makes sense to offer this as an input, just not...

Working on adding this to `tflint`, keep an eye out for a release soon: https://github.com/terraform-linters/tflint/pull/739 In addition to variables, it will also detect locals and and data sources that have...

Hmm, `v0.2.0`, which includes an ARM build, returns: ``` error getting credentials: No directory provided for file keyring ``` Running the `amd64` build works normally.

Yes, I've also run into this. Your notes/citations are accurate, this should be allowed. Short of disabling the rule entirely, it seems like walking the module tree and accumulating all...