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.

Sign up for GitHub

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

Jump to bottom

Change to CFLintAPI seems to have a negative impact on passing on values for quiet/verbose. #524

Closed
TheRealAgentK opened this issue Jan 2, 2018 · 1 comment
Closed

Change to CFLintAPI seems to have a negative impact on passing on values for quiet/verbose. #524

TheRealAgentK opened this issue Jan 2, 2018 · 1 comment
Assignees
@TheRealAgentK
Labels
bug
Milestone
1.4.0

Comments

@TheRealAgentK
Copy link
Collaborator

CFLintCLI.execute() now seems to use CFLintAPI internally. It seems that verbose and/or quiet are not properly passed down.

It manifests for me in me being unable to get the proper (set in main) values for quiet and verbose.

Will look into this in combination with #329.
@TheRealAgentK TheRealAgentK added this to the 1.3.1 milestone Jan 2, 2018
@TheRealAgentK TheRealAgentK self-assigned this Jan 2, 2018
@TheRealAgentK
Copy link
Collaborator Author

This is fixed and will be pushed in the feature branch for #329

TheRealAgentK added a commit to TheRealAgentK/CFLint that referenced this issue Jan 2, 2018
@TheRealAgentK
Fix for cflint#524, the setters for quiet and logError had an issue w…
5a6f279 
…hich caused them to not pass on the settings to the CFLint object.
@ryaneberly ryaneberly modified the milestones: 1.3.1, 1.4.0 May 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@TheRealAgentK TheRealAgentK

Labels
bug
Projects
None yet
Milestone
1.4.0
Development

No branches or pull requests
2 participants
@TheRealAgentK @ryaneberly