mysql-to-sqlite3 icon indicating copy to clipboard operation
mysql-to-sqlite3 copied to clipboard

Add option to connect to UNIX socket

Open ricklupton opened this issue 2 years ago • 2 comments

I wanted to connect to mysqld via a UNIX socket rather than IP, so I added an option. Hope this looks ok.

ricklupton avatar Apr 11 '22 16:04 ricklupton

Hi!

Thanks for the PR!!!

I've added some comments. Please resolve them.

Also, some tests using a socket would be nice :)

techouse avatar Apr 13 '22 15:04 techouse

Codecov Report

Merging #26 (9b253f2) into master (8558f1c) will increase coverage by 0.04%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #26      +/-   ##
==========================================
+ Coverage   89.81%   89.85%   +0.04%     
==========================================
  Files           7        7              
  Lines         422      424       +2     
==========================================
+ Hits          379      381       +2     
  Misses         43       43              
Impacted Files Coverage Δ
mysql_to_sqlite3/cli.py 95.55% <100.00%> (+0.10%) :arrow_up:
mysql_to_sqlite3/transporter.py 98.37% <100.00%> (+<0.01%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 8558f1c...9b253f2. Read the comment docs.

codecov[bot] avatar Apr 13 '22 15:04 codecov[bot]

@ricklupton can you please resolve the comments above?

techouse avatar Mar 12 '23 21:03 techouse

@techouse sorry for not responding to your comments!

I'm not working on this at the moment and don't have a local mysql database set up for testing at the moment. If it comes up again that I need this I'd be happy to try to add tests then but I'm afraid I don't have time at the moment.

If you want to complete this yourself and/or close it until/if I can look at it, please go ahead.

ricklupton avatar Apr 05 '23 11:04 ricklupton

I'm closing this then. Feel free to reopen when you have time again.

techouse avatar Apr 05 '23 12:04 techouse