8000 Check the root package for sniffs to install · Issue #20 · PHPCSStandards/composer-installer · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
8000

Check the root package for sniffs to install #20

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
Closed
Potherca opened this issue Mar 9, 2017 · 2 comments
Closed

Check the root package for sniffs to install #20

Potherca opened this issue Mar 9, 2017 · 2 comments
Assignees
@frenck
Labels
Type: feature request
Milestone
v0.4.0

Comments

@Potherca
Copy link
Member
Potherca commented Mar 9, 2017 •
edited
Loading

Problem/Motivation

As described in #19, the installer does not look in the root package. As it is completely viable for root packages to contain a code sniff, the installer should also check the root package.

Expected behaviour / Actual behaviour / Steps to reproduce

Not applicable.

Proposed changes

A separate step needs to be implemented that checks the root package to see if it contains any sniffs that (also) need installing.
@Potherca Potherca mentioned this issue Mar 9, 2017
Closed
@Potherca Potherca modified the milestone: v0.4.0 Mar 9, 2017
@Rarst Rarst mentioned this issue Apr 30, 2017
Merged
@frenck frenck mentioned this issue May 9, 2017
Merged
@frenck
@frenck frenck self-assigned this May 10, 2017
@frenck
Copy link
Contributor
frenck commented May 10, 2017

Merged into the master branch and will in the next release.

@frenck frenck closed this as completed May 10, 2017
@frenck frenck mentioned this issue May 10, 2017
Merged
@frenck frenck mentioned this issue May 31, 2017
Closed
@jrfnl jrfnl mentioned this issue Feb 4, 2022
Open
3 tasks
@jrfnl jrfnl mentioned this issue Apr 18, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@frenck frenck

Labels
Type: feature request
Projects
None yet
Milestone
v0.4.0
Development

No branches or pull requests
2 participants
@frenck @Potherca
0