Skip to content

Conversation

bcoe
Copy link
Contributor

@bcoe bcoe commented Mar 26, 2017

Adds support for restricting results to a scope, e.g., scope:types argument parser.

@coveralls
Copy link

coveralls commented Mar 26, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 886a8fe on scope into 32340f5 on master.

lib/search.js Outdated

if (params.scope) {
filters.must.push({
term: { 'package.scope': params.scope},
Copy link
Member

Choose a reason for hiding this comment

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

Can you add a space before {?

@coveralls
Copy link

coveralls commented Mar 26, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling ed0c3a6 on scope into 32340f5 on master.

@satazor satazor merged commit d17def2 into master Mar 26, 2017
@satazor satazor deleted the scope branch March 26, 2017 22:08
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