-
-
Notifications
You must be signed in to change notification settings - Fork 8
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
Update docs and config files - 1. #60
Conversation
terabytesoftw
commented
Apr 27, 2023
Q | A |
---|---|
Is bugfix? | ❌ |
New feature? | ✔️ |
Breaks BC? | ❌ |
PR Summary
|
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## master #60 +/- ##
===========================================
Coverage 100.00% 100.00%
Complexity 53 53
===========================================
Files 2 2
Lines 144 144
===========================================
Hits 144 144 ☔ View full report in Codecov by Sentry. |
/** | ||
* Database schema required by yiisoft/cache db for SQLite. | ||
*/ | ||
CREATE TABLE "cache" ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need these considering that we have DbHelper
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
They are in the documents as additional, they are not in the package as such.
Co-authored-by: Sergei Predvoditelev <sergei@predvoditelev.ru>
docs/en/migration/schema-mssql.sql
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need raw sql for drop table also.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@darkdef suggest removed because it is a bad practice, because you can mistakenly drop the table with data.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Its need for migrations.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can add it in another PR.
Co-authored-by: Sergei Predvoditelev <sergei@predvoditelev.ru>