Please upgrade here. These earlier versions are no longer being updated and have security issues.
HackerOne users: Testing against this community violates our program's Terms of Service and will result in your bounty being denied.
Options

Code Sniffer showing violations in default applications with Vanilla Standards

We have configured code sniffer with Vanilla Standards to make sure our custom code will be as per Vanilla Standards but it is showing violations even in default applications files i.e conversation?

I think, Vanilla Standards ruleset has been customized to make sure, it only shows violations when the code is not as per Vanilla/Garden framework but it proving me wrong as it also showing some violations in existing files.

I am not complaining here but a bit curious to know this behaviour.

Kindly throw some light here.

Comments

  • Options
    R_JR_J Ex-Fanboy Munich Admin

    The coding style has changed a few years ago. Changes weren't applied for every aspect, only for some reasonable things. This is a discussion which covers a lot of the developers questions back then: https://open.vanillaforums.com/discussion/30014/beware-the-4-spacening-this-week-25-may

    For the rest of the coding standard violations that you find, Lincs last comment would be good: "Fix 'em as you find 'em in code you change."

  • Options
    LincLinc Detroit Admin

    The dev team is currently discussing how to best update our standards to both streamline them and get our default code sniffing to only report the more relevant issues that need to be resolved.

Sign In or Register to comment.