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.

Sign up for GitHub

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

Jump to bottom

Import sqlite adapter #671

Merged
merged 7 commits into from
Dec 31, 2023
Merged

Import sqlite adapter #671

merged 7 commits into from
Dec 31, 2023

Conversation

markstory
Copy link
Member

  • Import sqlite driver code from phinx.
  • Update config parsing to use ConnectionManager as that is the eventual end state.
  • Fix strict errors caused by uninitialized properties.
  • Fix configuration parsing to unskip mysql tests.

@markstory
Import sqlite driver and tests
4fe83fd 
- Import sqlite driver code from phinx.
- Update config parsing to use ConnectionManager as that is the eventual
  end state.
- Fix strict errors caused by uninitialized properties.
@markstory
Fix skipped mysql tests and missing property initialization
1042af9
@markstory
Fix psalm errors.
145a508
@markstory
Update phpstan baseline
97c1c1a
@markstory
Fix phpcs
767f4f2
@markstory
Align default encoding with test expectations.
b528427
@markstory
Fixate the default encoding
04494d3 
not all mysql instances use utf8mb4 as their default encoding.
@markstory markstory merged commit fb8e05f into no-phinx Dec 31, 2023
9 of 10 checks passed
@markstory markstory deleted the import-sqlite branch December 31, 2023 03:31
@markstory markstory linked an issue Jan 4, 2024 that may be closed by this pull request
4.x Migrations without Phinx #647
Closed
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

4.x Migrations without Phinx
1 participant
@markstory