Skip to content

Small fix to enable mysql schema creation#396

Closed
naturalethic wants to merge 1 commit intolldap:mainfrom
naturalethic:fix-mysql-schema
Closed

Small fix to enable mysql schema creation#396
naturalethic wants to merge 1 commit intolldap:mainfrom
naturalethic:fix-mysql-schema

Conversation

@naturalethic
Copy link
Copy Markdown

These two small changes will fix this to work with mysql. I do not know if it breaks sqlite.

Copy link
Copy Markdown
Member

@nitnelave nitnelave left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes look good, but I'd like to test them with all of sqlite, mysql and postgresql before merging.

@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 16, 2022

Codecov Report

Merging #396 (1688994) into main (c64d32e) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #396   +/-   ##
=======================================
  Coverage   67.99%   68.00%           
=======================================
  Files          39       39           
  Lines        6353     6354    +1     
=======================================
+ Hits         4320     4321    +1     
  Misses       2033     2033           

martadinata666 added a commit to martadinata666/lldap that referenced this pull request Jan 8, 2023
martadinata666 added a commit to martadinata666/lldap that referenced this pull request Jan 9, 2023
martadinata666 added a commit to martadinata666/lldap that referenced this pull request Jan 11, 2023
martadinata666 added a commit to martadinata666/lldap that referenced this pull request Jan 11, 2023
martadinata666 added a commit to martadinata666/lldap that referenced this pull request Jan 11, 2023
@nitnelave
Copy link
Copy Markdown
Member

Closing in favor of #418

@nitnelave nitnelave closed this Jan 13, 2023
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

Successfully merging this pull request may close these issues.

2 participants