Skip to content
This repository has been archived by the owner on May 17, 2021. It is now read-only.

COMPASS-704 Remove packages option #93

Merged
merged 2 commits into from
Jul 13, 2017
Merged

Conversation

pzrq
Copy link
Contributor

@pzrq pzrq commented Jul 13, 2017

It never worked properly, workaround in Compass is to put tests into the top level tests folder, or better yet to convert the plugin to an external plugin.

It never worked properly, workaround in Compass is to put tests into the top level tests folder, or better yet to convert the plugin to an external plugin.
@pzrq
Copy link
Contributor Author

pzrq commented Jul 13, 2017

The command injection security issue is well upstream but hopefully will see a bug fix release soon: tj/node-growl#62

@rueckstiess rueckstiess self-requested a review July 13, 2017 06:51
Copy link
Member

@rueckstiess rueckstiess left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the cleanup, LGTM

@pzrq pzrq merged commit fa9cd55 into master Jul 13, 2017
@pzrq pzrq deleted the COMPASS-704-remove-packages-option branch July 13, 2017 06:51
@pzrq
Copy link
Contributor Author

pzrq commented Jul 13, 2017

Thanks @rueckstiess 👍

@pzrq
Copy link
Contributor Author

pzrq commented Jul 13, 2017

Published in v13.0.11.

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

Successfully merging this pull request may close these issues.

2 participants