Thanks for your PR. I don't think this is harmful or could break anything, but it seems a bit redundant.

Can you try this alternate approach?

In theinit/actions.yml file adddefault: "${{ env.CODEQL_THREADS }}" to thethreads input. This should be sufficient.

Yep I can give this a shot. My initial change was just trying to match up the way the ram variable was treated

Yep I can give this a shot. My initial change was just trying to match up the way the ram variable was treated

Great. Let me know if this works for you. Now that I'm engaged, I'll be able to respond more quickly to this PR.

@aeisenberg lmk what you think 👍

@aeisenberg ah I don't think you can use env variable expressions in inputs

Error: /Users/runner/work/codeql-action/codeql-action/./../action/init/action.yml (Line: 118, Col: 14): Unrecognized named-value: 'env'. Located at position 1 within expression: env.CODEQL_THREADS

Oh...that's really unfortunate. Apologies for the confusion. I'm asking internally what's going on since this is a pattern that'sbeing used elsewhere.

Oh...that's really unfortunate. Apologies for the confusion. I'm asking internally what's going on since this is a pattern that'sbeing used elsewhere.

Most of those examples commented out thedefault field with a# though. I'm not sure why theenv context is not allowed, perhaps it is not available until a workflow starts to "run".

In any case it probably makes sense to have the threads and ram behavior configuration be exactly the same so there aren't any surprises

Thanks for the contribution@austinpray-mixpanel! Could you follow the instructions inhttps://github.com/github/codeql-action/blob/main/CONTRIBUTING.md#common-tasks to rebuild the project?

@henrymercer done 👍

Thank you all!