You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When considering which files to sync, it would be useful to have an include flag as well as the current ignore 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 use sml-sync mostly to develop Python.
The text was updated successfully, but these errors were encountered:
Thanks @albertofavaro - just to check, what would you expect the behaviour to be in the following cases:
--include="*.py" --ignore="secrets.py"
--include="test.py, test2.py" --ignore="test.py"
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 --include is a filter that gets applied before the flag --ignore. With this logic, neither of the examples that @janfreyberg put forward would result in an error. They would be interpreted as:
All .py files with the exception of secrets.py are considered.
The file test2.py is considered, whereas test.py is not.
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: