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

exclude all file at higher level #77

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

liorsion
Copy link

This is not a complete solution as it doesn't support flips and a few other options, however - it is at a higher level position and so automatically supports all actions and there's no need to duplicate this code. If it makes sense I'll add the rest of the code support.

@MathieuLoutre
Copy link
Owner

Is this to exclude files from Grunt when uploading? If that's the case, have you tried using ! in your globbing pattern? It's supported natively by Grunt: http://gruntjs.com/api/grunt.file#globbing-patterns

If that's not what it's for, then I might need a use case to consider this. Thanks!

@liorsion
Copy link
Author

It's used to exclude everything - since the exclude exists in the docs and in the download part but not everywhere. Personally I used it for the upload, and the '!' didn't work for me.

btw, the use case is if you want to set different cache headers to specific files (not directories).

@MathieuLoutre
Copy link
Owner

Interesting that the ! didn't work for you. Sorry that I'm only taking another look now, I've had very little time to focus on my open source projects in the past few months. If you're still happy to add the rest of the code support, I'd be glad to merge this.

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.

2 participants