Results 125 comments of Rob Nelson

@arlt It would be nice for this to parse properly, which is something we can look at later. But I don't think it has high value, given the other style...

This is specifically referring to [lexer.rb's KNOWN_TOKENS](https://github.com/rodjek/puppet-lint/blob/master/lib/puppet-lint/lexer.rb#L91) which includes `:TYPE` with a hard-coded list of known types. The list is not exhaustive. Any variable declaration that is preceeded by a...

Just for comparison's sake, do you see the same slow down with puppet parser validate and rspec-puppet, or is it only puppet-lint? -- Rob Nelson

It looks like you want a control comment to turn off all checks, correct? Can you also provide file contents that caused the linter to crash? I think we would...

@trevor-vaughan does your crash look the same as in #578?

Can one of you provide a reproduction, what you expect, and the actual results you received? I want to make sure we are clear on the problem and expectation.

Ah. It sounds like you are asking for two things then: * The check `single_quote_string_with_variables` to be dropped from `error` to `warning` level * The check `single_quote_string_with_variables` to parse `\$`...

@digital-shokunin thanks for the great detail! We can definitely work off that information. In the meantime, can you try this flag with warning for the value? It's not perfect, but...

It's at least a big enough deal to make this a decidedly major version bump. I'd like to discuss this more on a triage call and then add some notes...

In yesterday's triage, we discussed the need to support AppOrchestration code in the API, with the code that @binford2k provided. I've added this to the PR. We also discussed the...