Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test Harry's Sublime/CFLint instructions and i18n to English if ok #307

Closed
TheRealAgentK opened this issue Jun 26, 2017 · 9 comments
Closed
Assignees

Comments

@TheRealAgentK
Copy link
Collaborator

https://github.com/cflint/CFLint/wiki/Installation-von-CFLint-&-Sublime-Linter-Plugin

@KamasamaK
Copy link
Collaborator

When translated, it became clear that the instructions were for an older version of CFLint. I have updated the instructions to work with the current version of both CFLint and the SublimeLinter plugin. I have started to update the plugin to work with the JAR file, and will update the instructions accordingly when finished.

@TheRealAgentK
Copy link
Collaborator Author

@KamasamaK Awesome - a couple of people in the cflint channel on the CFML Slack team have started to look into that, too. Not sure if you're on there, but it might be worth coordinating with them.

@KamasamaK
Copy link
Collaborator

My changes have already been merged to SublimeLinter-contrib-CFLint including documentation updates. Once the maintainer creates a new release, the wiki page will be outdated and largely unnecessary, but I will revisit at that time. Either way, I think this issue can be closed.

@TheRealAgentK
Copy link
Collaborator Author

@KamasamaK Awesome, thanks. @derrick13 - would you mind having a look at the updated repo @KamasamaK mentioned and confirm that which those instructions and their codebase the setup works?

@HarryMuc
Copy link

@agentk Unfortunately after Sublime updated my CFLint SublimeLinter plugin it stopped working. The configfile is not found, even if I define "aux_config_dirs".
It would be great if someone could share a working example (on windows).
I would enable these rules only: "MISSING_VAR,OUTPUT_ATTR,AVOID_USING_STRUCTNEW,AVOID_USING_ARRAYNEW,UNUSED_LOCAL_VARIABLE"

This is my configuration atm (after several tries):

               "cflint": {
                "@disable": false,
                "args": [],
                "aux_config_dirs": [
                    "D:/projekte_cf/contens/contens5",
                    "D:/projekte_cf/contens/contens5/",
                    "D:/projekte_cf/contens/contens5/contens",
                    "D:/projekte_cf/contens/contens5/contens/"
                ],
                "config_file_name": "cflintrc.xml",
                "excludes": [],
                "jar_file": "D:/CFLint/CFLint-1.1.0-all.jar"
            },

And this is my minimal configuration file "cflintrc.xml":

<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
<config>
    <rule name="MISSING_VAR"></rule>
</config>

@TheRealAgentK
Copy link
Collaborator Author

@KamasamaK Help? :-)

@KamasamaK
Copy link
Collaborator

I've confirmed with @derrick13 on Slack that it's not the case that "The configfile is not found". The config file was loaded, but not working as he expected. We could only get the <excludes code="RULE_CODE" /> tags to work. This is beyond the scope of the plugin. He has also been informed that the XML format being used is being deprecated.

@TheRealAgentK
Copy link
Collaborator Author

Not sure where to go from here @ryaneberly

@TheRealAgentK
Copy link
Collaborator Author

I close this as we seem to have a working Sublime integration again now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants