jonathanunderwood

@YannickJadoul yep, I figured that out after posting my comment. I worked around that, and then hit further inconsistencies in how environment variables are passed through to the build: issue...

The docs list "-' as an allowed character in Role names: http://docs.aws.amazon.com/IAM/latest/UserGuide/reference_iam-limits.html

@nemesisdesign How do you recommend dealing with a UCI parameter that can either be a string or a list? One such example is the `proto` parameter in a firewall `rule`....

> @nemesisdesign How do you recommend dealing with a UCI parameter that can either be a string or a list? One such example is the `proto` parameter in a firewall...

@nemesisdesign currently qa checks are failing with this: ``` ./netjsonconfig/backends/openwrt/converters/firewall.py:200:5: C901 'Firewall.__netjson_redirect' is too complex (14) ``` I could split `__netjson_redirect()` up, but it seems like a pointless exercise, since...

> @nemesisdesign currently qa checks are failing with this: > > ``` > ./netjsonconfig/backends/openwrt/converters/firewall.py:200:5: C901 'Firewall.__netjson_redirect' is too complex (14) > ``` > > I could split `__netjson_redirect()` up, but...

> Thanks @jonathanunderwood! I'm looking to find a chance to test your work this week. Great! There's still a LOT of work to do, though. Test coverage is currently incomplete,...

@nemesisdesign any idea what's going on with the test execution - travis/coveralls seems to have been waiting forever?

It's `tool` not `tools` that you need. E.g. ```yaml [tool.sqlfluff.core] dialect = "snowflake" templater = "dbt" runaway_limit = 10 max_line_length = 100 indent_unit = "space" [tool.sqlfluff.templater.dbt] project_dir = "./proj" [tool.sqlfluff.indentation]...

@ValerioBob Sadly I ran out of time to work on this back in 2021. There's actually not that much left to do, so please feel free to dig in and...