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

Dev #202

Merged
merged 38 commits into from
Oct 23, 2016
Merged

Dev #202

merged 38 commits into from
Oct 23, 2016

Conversation

ryaneberly
Copy link
Contributor

No description provided.

ryaneberly and others added 30 commits July 28, 2016 21:28
Conflicts:
	src/main/java/com/cflint/plugins/core/VariableNameChecker.java
Prefilter (do not parse) files that are excluded by the filter.

trim the file name.
…name (cfquery, cfloop, and cfinvoke)

Fixed BugInfo to trim file names
Fix cfml tag checker to verify the cfmlTag was not null before checking
Added Test for cfelement variable nameEndsInNumberScript
Added new Argument Default condition checker for variables that don't have a default and the code is checking if the argument exists or isdefined
# Conflicts:
#	src/main/java/com/cflint/tools/CFLintFilter.java
Merger from Head Master
# Conflicts:
#	pom.xml
#	src/main/java/com/cflint/CFLint.java
#	src/main/java/com/cflint/ant/CFLintTask.java
#	src/main/java/com/cflint/main/CFLintMain.java
Added access to the tokenstream in the Context.
Added support for 
//cflint ignore:line

and 
//cflint ignore:MISSING_VAR
honor includes of rules with more than 1 message.
Includes and excludes from config xml file are now marged in the filter used when populating bug messages
Add support for @CFLintIgnore to component and function declarations.

Plus some cleanup, and better support for nested contexts.
Add support for @CFLintIgnore to cfcomponent and cffunction tags.

cleanup bugs - all tests passing.
Conflicts:
	src/main/java/com/cflint/CFLint.java
	src/main/java/com/cflint/plugins/Context.java
@ryaneberly ryaneberly merged commit cc45d56 into master Oct 23, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants