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

including the option to check if sqlite3 has been installed #272

Merged
merged 1 commit into from
Dec 19, 2018

Conversation

SedaArat
Copy link
Contributor

No description provided.

@remram44 remram44 added the type:enhancement Propose enhancement to the lesson label Dec 12, 2018
@remram44
Copy link
Contributor

Hi @SedaArat, thanks for contributing! I find that simply running sqlite3 is enough. Provided the user knows how to get to a terminal...

@SedaArat
Copy link
Contributor Author

That's right. Thank you for your prompt response. Yet, it does not give you the path back. It may or may not be very important for the user. I just wanted to include something (some code) that once can check if it is already installed.

@henrykironde
Copy link
Contributor

I think both ways are fine. This looks good to me(LGTM) since it is Unix. sqlite3 --version could also do.
I feel users will get more familiar with which.

@henrykironde henrykironde merged commit 94686e0 into swcarpentry:gh-pages Dec 19, 2018
fmichonneau pushed a commit to fmichonneau/sql-novice-survey that referenced this pull request Nov 1, 2019
Remove root and layout from YAML for boilerplate
zkamvar pushed a commit that referenced this pull request May 8, 2023
including the option to check if sqlite3 has been installed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:enhancement Propose enhancement to the lesson
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants