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.

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

Move database schema to migration #352

Merged
merged 1 commit into from Jul 10, 2017

Conversation

Projects
None yet
2 participants
@nickvergessen
Copy link
Member

nickvergessen commented Jul 7, 2017

To test just install the app on a fresh instance and check that the database tables look the same.

@nickvergessen nickvergessen added this to the 3.0 (Nextcloud 13) milestone Jul 7, 2017

@nickvergessen nickvergessen requested a review from Ivansss Jul 7, 2017

@nickvergessen nickvergessen referenced this pull request Jul 8, 2017

Merged

Participant types #353

7 of 9 tasks complete
Move database schema to migration
Signed-off-by: Joas Schilling <coding@schilljs.com>

@nickvergessen nickvergessen force-pushed the migrations branch from 33ea2c5 to 2ab09a6 Jul 10, 2017

@codecov

This comment has been minimized.

Copy link

codecov bot commented Jul 10, 2017

Codecov Report

Merging #352 into master will decrease coverage by 0.77%.
The diff coverage is 0%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #352      +/-   ##
============================================
- Coverage     15.23%   14.45%   -0.78%     
- Complexity      276      280       +4     
============================================
  Files            18       19       +1     
  Lines          1326     1397      +71     
============================================
  Hits            202      202              
- Misses         1124     1195      +71
Impacted Files Coverage Δ Complexity Δ
lib/Migration/Version2000001Date20170707093535.php 0% <0%> (ø) 4 <4> (?)

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 b0a8cab...2ab09a6. Read the comment docs.

@Ivansss Ivansss merged commit 91e21d7 into master Jul 10, 2017

1 of 3 checks passed

codecov/patch 0% of diff hit (target 15.23%)
Details
codecov/project 14.45% (-0.78%) compared to b0a8cab
Details
continuous-integration/drone/push the build was successful
Details

@Ivansss Ivansss deleted the migrations branch Jul 10, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.