Skip to content

Rules disabled by default #8

@bockensm

Description

@bockensm

I have prepared rules for preferring member function syntax over the older functions (e.g. someVar.append() instead of arrayAppend(). The problem is there are a lot of rules and it bogs down performance significantly. The rules are based on what I found on CFDocs: https://cfdocs.org/member. I broke them out by variable type (array, query, xml, etc) so a user would only need to exclude a handful of categories if they don't want to run these.

I like having as many options as possible to cover most cases, but something like member function syntax is going to raise a high number of flags on older code. It'll also provide limited value to users of CF10 or earlier.

So, the questions:

  1. Should the package be concerned with users of old versions of ColdFusion, and how far back? Language support is not mentioned anywhere.
  2. Are rules for preferring member functions desirable?
  3. There is a facility to exclude entire files, but that would be a full disable, not an opt-out. Should the strategy continue to be "run all unless told otherwise", even if that degrades performance?

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions