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

Added status to migration steps. #7

Merged
merged 1 commit into from
Oct 15, 2018
Merged

Conversation

triadev
Copy link
Owner

@triadev triadev commented Oct 14, 2018

No description provided.

@triadev triadev self-assigned this Oct 14, 2018
@coveralls
Copy link

coveralls commented Oct 14, 2018

Pull Request Test Coverage Report for Build 136

  • 27 of 27 (100.0%) changed or added relevant lines in 5 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.1%) to 95.878%

Totals Coverage Status
Change from base Build 133: 0.1%
Covered Lines: 628
Relevant Lines: 655

💛 - Coveralls

@codecov-io
Copy link

codecov-io commented Oct 14, 2018

Codecov Report

Merging #7 into master will increase coverage by 0.13%.
The diff coverage is 93.96%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master       #7      +/-   ##
============================================
+ Coverage      95.5%   95.64%   +0.13%     
- Complexity      197      203       +6     
============================================
  Files            35       36       +1     
  Lines           668      689      +21     
============================================
+ Hits            638      659      +21     
  Misses           30       30
Impacted Files Coverage Δ Complexity Δ
src/Exception/MigrationsNotExist.php 100% <100%> (ø) 1 <1> (?)
...Business/Repository/ElasticsearchMigrationStep.php 100% <100%> (ø) 8 <8> (?)
src/ElasticsearchMigrationDatabase.php 97.53% <100%> (ø) 54 <0> (ø) ⬇️
src/ElasticsearchMigration.php 100% <100%> (ø) 17 <0> (ø) ⬇️
...s/Repository/ElasticsearchMigrationStepReindex.php 100% <100%> (ø) 3 <2> (?)
...sitory/ElasticsearchMigrationStepDeleteByQuery.php 100% <100%> (ø) 4 <2> (?)
...sitory/ElasticsearchMigrationStepUpdateByQuery.php 100% <100%> (ø) 6 <2> (?)
...pository/ElasticsearchMigrationStepCreateIndex.php 100% <100%> (ø) 5 <2> (?)
...pository/ElasticsearchMigrationStepUpdateIndex.php 100% <100%> (ø) 7 <2> (?)
...ess/Repository/ElasticsearchMigrationStepAlias.php 100% <100%> (ø) 3 <2> (?)
... and 10 more

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 7781bf2...b5b29d0. Read the comment docs.

@coveralls
Copy link

coveralls commented Oct 14, 2018

@triadev triadev force-pushed the migration_step_status branch 3 times, most recently from cf09fb9 to 51194ce Compare October 14, 2018 17:51
@triadev triadev merged commit 1cd1f25 into master Oct 15, 2018
@triadev triadev deleted the migration_step_status branch October 21, 2018 12:00
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.

None yet

3 participants