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

Jump to bottom

Check SensioLabsInsight report and remove alerts #6062

Open
mattab opened this issue Aug 25, 2014 · 3 comments
Open

Check SensioLabsInsight report and remove alerts #6062

mattab opened this issue Aug 25, 2014 · 3 comments
Labels
Task Indicates an issue is neither a feature nor a bug and it's purely a "technical" change.
Milestone
Backlog (Help wan...

Comments

@mattab
Copy link
Member

mattab commented Aug 25, 2014

SensioLabsInsight continuously analyze PHP code to detect security risks, find bugs, provide actionable metrics and help improve your projects in the long term.

The Piwik project is analysed here: https://insight.sensiolabs.com/projects/2fcaa00c-b158-44ee-898d-96f49d8ac63c

The goal of this ticket is to check the alerts and remove them.

Currently there are 2755 violations of rules. Most of them are false positive.... When they are false positive, when I'm logged in, I can "ignore" these alerts or specific alerts so they don't display.
@mattab mattab added this to the Mid term milestone Aug 25, 2014
@mattab mattab added the Task label Aug 25, 2014
@ThaDafinser
Copy link
Contributor

@mattab dont you want to remove the library dependencies on thecheck? So you can start on piwik coding issues and not from external librarys...

@mattab
Copy link
Member Author

mattab commented Aug 27, 2014

I looked quickly in the UI but can't find option to ignore given path, do you know if it's possible?

@ThaDafinser
Copy link
Contributor

@mattab https://insight.sensiolabs.com/faq#how-can-I-ignore-directories-in-analysis

@mattab mattab modified the milestones: Mid term, Long term Oct 11, 2014
@mattab mattab modified the milestones: Long term, Mid term Dec 23, 2015
@mattab mattab modified the milestones: Long term, Mid term Dec 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Task Indicates an issue is neither a feature nor a bug and it's purely a "technical" change.
Projects
None yet
Milestone
Backlog (Help wanted)
Development

No branches or pull requests
2 participants
@mattab @ThaDafinser