-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add include
flag
#11
Comments
Thanks @albertofavaro - just to check, what would you expect the behaviour to be in the following cases:
I think 1. should work and the ignore should supersede the include, and 2. should raise an error, but would be good to know what you expect to happen. I think given that you can specify ignore arguments in the config, it could be easy to have clashes. |
Thanks for raising this! Once we've settled on a behaviour, happy to walk you -- or anyone else interested in working on this -- through the steps to get this implemented! |
I don't have a strong opinion. I would suggest that the flag
|
When considering which files to sync, it would be useful to have an
include
flag as well as the currentignore
flag. The list of file types that I plan to include is quite minimal (.py
,.md
and perhaps.sh
). Having an include flag would cater for people who usesml-sync
mostly to develop Python.The text was updated successfully, but these errors were encountered: