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

Strict Glob Support #284

Merged
merged 1 commit into from Dec 2, 2014
Merged

Strict Glob Support #284

merged 1 commit into from Dec 2, 2014

Conversation

josh
Copy link
Contributor

@josh josh commented Dec 1, 2014

Glob syntax must end in /* or /**/*.

/cc @rafaelfranca

@rafaelfranca
Copy link
Member

:shipit:

josh added a commit that referenced this pull request Dec 2, 2014
@josh
Merge pull request #284 from rails/strict-glob
43d59da 
Strict Glob Support
@josh josh merged commit 43d59da into master Dec 2, 2014
@josh josh deleted the strict-glob branch December 2, 2014 16:05
@alex-quiterio
Copy link

Hi @josh and @rafaelfranca.

I don't understand why we must be restricted to the /* or /**/*/ syntax. With that approach we are limited from what we can specify for each sub-directory. For example: /**/target_file.scss is not allowed with this current implementation. It might be good to have something extra on the syntax to allow targeting files from sub-directories right?

Thanks for the clarification on this.

@ruipserra ruipserra mentioned this pull request Jan 7, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@josh @rafaelfranca @alex-quiterio