Jesse Newland

Oh wow, I had never seen Shellwords before. Amazing. This would definitely address a long-standing wart in God. I'm sure there are edge cases in Shellwords, though. Unsure how to...

:+1:

I'm encountering a similar error with another public module: ``` - Updating terraform-aws-modules/iam/aws (from )…bundler: failed to load command: ./dependabot.rb (./dependabot.rb) /usr/local/bundle/bundler/gems/dependabot-core-44f458dba8ce/terraform/lib/dependabot/terraform/file_updater.rb:25:in `block in updated_dependency_files' /usr/local/bundle/bundler/gems/dependabot-core-44f458dba8ce/terraform/lib/dependabot/terraform/file_updater.rb:21:in `each' /usr/local/bundle/bundler/gems/dependabot-core-44f458dba8ce/terraform/lib/dependabot/terraform/file_updater.rb:21:in `updated_dependency_files' /usr/src/app/dependabot.rb:131:in...

```terraform module "irsa_aws_alb_ingress_controller" { source = "terraform-aws-modules/iam/aws//modules/iam-assumable-role-with-oidc" version = "~> 2.18.0" create_role = true role_name = "aws-alb-ingress-controller" provider_url = replace(module.eks.cluster_oidc_issuer_url, "https://", "") role_policy_arns = [aws_iam_policy.aws_alb_ingress_controller.arn] oidc_fully_qualified_subjects = ["system:serviceaccount:default:aws-alb-ingress-controller"] } ```

Following up on this, the [existing documentation on auto_increment](https://schemahero.io/docs/managing-tables/attributes/) is a bit confusing: it shows an example schema for Postgres that uses the seemingly incompatible `autoIncrement` attribute while mentioning the...

Here's scan output for a Sonos Move that's currently appearing without an ignore link: ``` 2021-11-27 02:21:34 DEBUG [pyatv.core.scan]: Auto-discovered move at 192.168.4.81:7000 via Protocol.AirPlay ({'acl': '0', 'deviceid': '48:A6:B8:EE:98:16', 'features':...

> If I prepare a branch with additional log points, can you try it out for me and report the result back? Yes, happily! However, I haven't seen any new...

Hey, thanks for the contribution! I think this would be a pretty neat additional feature, but believe that the way this is implemented now may break this functionality for people...

> Maybe a list of the names of CRDs as "excluded" in the kubevalidator.yaml? This sounds like a good approach for those that would prefer to ignore CRDs rather than...

> I think there is no more need for this PR. I'm using this PR to support [creating a GitHub App from a manifest](https://docs.github.com/en/rest/apps/apps#create-a-github-app-from-a-manifest), which expects the integrator to handle...