Skip to content

Fixes UserFrosting#965 - Ignore files that don't end in .php#998

Merged
lcharette merged 2 commits intouserfrosting:hotfixfrom
xrobau:fix-database-migration
Jun 13, 2019
Merged

Fixes UserFrosting#965 - Ignore files that don't end in .php#998
lcharette merged 2 commits intouserfrosting:hotfixfrom
xrobau:fix-database-migration

Conversation

@xrobau
Copy link
Copy Markdown
Contributor

@xrobau xrobau commented Jun 9, 2019

This ignores any files in Migrations that do not end in .php

It causes problems when scripts, or data, or documentation is
left in the Migrations directory, and the migration task assumes
that they are PHP Classes and crashes.

Signed-Off-By: Rob Thomas xrobau@gmail.com

This ignores any files in Migrations that do not end in .php.
It causes problems when scripts, or data, or documentation is
left in the Migrations directory, and the migration task assumes
that they are PHP Classes and crashes.

Signed-Off-By: Rob Thomas <xrobau@gmail.com>
@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 11, 2019

Codecov Report

Merging #998 into hotfix will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##             hotfix     #998      +/-   ##
============================================
+ Coverage     68.21%   68.22%   +<.01%     
- Complexity     2139     2140       +1     
============================================
  Files           159      159              
  Lines          7243     7244       +1     
============================================
+ Hits           4941     4942       +1     
  Misses         2302     2302
Impacted Files Coverage Δ Complexity Δ
...es/core/src/Database/Migrator/MigrationLocator.php 100% <100%> (ø) 5 <0> (+1) ⬆️

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 1b2f828...e32313e. Read the comment docs.

@lcharette lcharette merged commit a4e4b81 into userfrosting:hotfix Jun 13, 2019
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.

2 participants