Skip to content

Move Atk4\Dsql under Atk4\Data\Persistence\Sql namespace#879

Merged
DarkSide666 merged 2 commits intodevelopfrom
move_dsql_into_persistence_sql
May 25, 2021
Merged

Move Atk4\Dsql under Atk4\Data\Persistence\Sql namespace#879
DarkSide666 merged 2 commits intodevelopfrom
move_dsql_into_persistence_sql

Conversation

@mvorisek
Copy link
Copy Markdown
Member

@mvorisek mvorisek commented May 12, 2021

I simply moved the files, replaced Atk4\Dsql with Data\Persistence\Sql on string level and adjusted composer.json and phpunit/phpstan config files.

@mvorisek mvorisek requested a review from georgehristov May 12, 2021 14:32
@mvorisek mvorisek requested a review from DarkSide666 May 12, 2021 14:45
@mvorisek mvorisek force-pushed the move_dsql_into_persistence_sql branch from 8f0975f to 9a09123 Compare May 12, 2021 14:48
@mvorisek mvorisek changed the title Move Atk4\Dsql under Atk4\Sql\Persistence\Sql namespace Move Atk4\Dsql under Atk4\Data\Persistence\Sql namespace May 13, 2021
@mvorisek mvorisek force-pushed the move_dsql_into_persistence_sql branch from 9b5f14e to 5b793b4 Compare May 13, 2021 14:52
@mvorisek
Copy link
Copy Markdown
Member Author

@georgehristov please approve

is there any simple naming adjustment you want to have in this PR?

Copy link
Copy Markdown
Member

@DarkSide666 DarkSide666 left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

@DarkSide666 DarkSide666 merged commit 54cd831 into develop May 25, 2021
@DarkSide666 DarkSide666 deleted the move_dsql_into_persistence_sql branch May 25, 2021 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants