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

Clarity / documentation #66

Open
eileenmcnaughton opened this issue May 4, 2021 · 0 comments
Open

Clarity / documentation #66

eileenmcnaughton opened this issue May 4, 2021 · 0 comments

Comments

@eileenmcnaughton
Copy link
Contributor

Just noting that I found I was able to do

amp config:set --mysql_dsn=mysql:://root@database:3306

but

a) it's not in the read me and
b) it only appeared to work AFTER I'd run config::set with db_type

I think it makes sense to update the readme if that is the case

Also the 'pre-alpha' part of the readme is wrong IMHO given we use this daily

@totten

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

No branches or pull requests

1 participant