-
Notifications
You must be signed in to change notification settings - Fork 50
Revamp CLI user experience #388
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
base: main
Are you sure you want to change the base?
Conversation
tensorlake auth login -> tensorlake login tensorlake auth status -> whoami
cdf9907
to
eadb9fd
Compare
9775df2
to
96587ce
Compare
src/tensorlake/cli/_configuration.py
Outdated
LEGACY_CREDENTIALS_PATH = CONFIG_DIR / "credentials.json" | ||
|
||
# Local project configuration file | ||
LOCAL_CONFIG_FILE = Path.cwd() / ".tensorlake.toml" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should we make this into a directory, .tensorkale/config.toml
? It will probably be easier to make any changes in the future if we need to that way.
If I look at this from past experiences, I like what GitHub did (all config files inside a directory from the start), more than what we did at Netlify (one file in the root, and then adding a directory later when we needed to dump more files in the repo). 🙈
349d17f
to
4104200
Compare
Fixes: #385