search

russplaysguitar / navel

Working towards a linter for cfscript
MIT License
2 stars 0 forks source link

component output="false" #19

Open russplaysguitar opened 11 years ago

russplaysguitar commented 11 years ago

component output="false" should be set for components, else lint warning

cfjedimaster commented 11 years ago

If I remember right, it is only a big deal for tag based CFCs? I think script based ones auto assume output=false, or don't have the 'whitespace generating like a mofo' issue.

atuttle commented 11 years ago

it's just a warning. can't hurt.

russplaysguitar commented 11 years ago

if script-based components assume output=false, then we could skip this rule, right? anyone know for sure (Railo4 and CF10)? i might have to test that out later...

cfjedimaster commented 11 years ago

We shouldn't skip it - just lint it only in tag-based components.

russplaysguitar commented 11 years ago

fyi i plan on keeping this project focused on a cfscript linter only (and not a tag linter) in order to limit the scope and prevent the duality problems associated with supporting both tags and script.