Morgante Pell

Results 276 comments of Morgante Pell

This seems like a reasonable suggestion, happy to review a PR adding it.

This looks like a minor bug in how we compute the index length when importing. We probably need to use `max(length(google_kms_crypto_key.key), length(var.keys))`.

@cakeben A PR to fix this would be appreciated if you have a chance.

@nick4fake I fixed issue 4 via #18, FYI.

It looks like this was originally meant to help with building a bridge, but has since been removed. We should remove the variable.

@Ashu999 Tests and linters are failing. Please make sure everything passes.

> I think this is going to break the studio in the web app, where we rely on this constant to represent the pattern file. Yeah we'll need to do...