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

Missing MySQL dependency #57

Closed
benstadlbauer opened this issue Jan 7, 2024 · 4 comments
Closed

Missing MySQL dependency #57

benstadlbauer opened this issue Jan 7, 2024 · 4 comments
Labels
good first issue A good issue for beginners help wanted

Comments

@benstadlbauer
Copy link

This issue is more of a question/clarification.

Why is the MySQL-driver not listed as dependency in the complete project?

If one completes the guide Accessing data with MySQL then there is no problem (as @Buzzardo mentioned here), because the guide states to include the MySQL Driver (see Starting with Spring Initializr).

But if one directly downloads this git repository and tries to run the complete project, it won't work. There are already two issues regarding this very problem: #53 , #52 .

Thank you!

@Buzzardo
Copy link
Contributor

Buzzardo commented Jan 8, 2024

I suppose we could add a note at the top to state that MySQL must be available. I'll make this one available to the community, as we are trying to encourage more community engagement with the guide repositories.

@Buzzardo Buzzardo added help wanted good first issue A good issue for beginners labels Jan 8, 2024
@j0holo
Copy link
Contributor

j0holo commented Apr 21, 2024

I have a similar problem. The documentation is no longer up-to-date.

In build.gradle I need to add the following dependency: runtimeOnly 'com.mysql:mysql-connector-j'

And in application.properties the spring.datasource.driver-class-name=com.mysql.cj.jdbc.Driver setting is no longer required to start the server.

Could I create a pull request for these changes? Or is there something I'm missing?

@Buzzardo
Copy link
Contributor

Feel free to create a pull request. Thanks.

@robertmcnees
Copy link
Contributor

Fixed with #58

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue A good issue for beginners help wanted
Projects
None yet
Development

No branches or pull requests

4 participants