Skip to content

Conversation

@Iandenh
Copy link
Contributor

@Iandenh Iandenh commented Oct 16, 2017

No description provided.

$this->assertRegExp('/\s*up 20120103083300 2012-01-11 23:53:36 2012-01-11 23:53:37 *\*\* MISSING \*\*'.PHP_EOL.
'\s*up 20120815145812 2012-01-16 18:35:40 2012-01-16 18:35:41 Example *\*\* MISSING \*\*'.PHP_EOL.
'\s*down 20120111235330 TestMigration'.PHP_EOL.
$this->assertRegExp('/\s*up 20120103083300 2012-01-11 23:53:36 2012-01-11 23:53:37 *\*\* MISSING \*\*' . PHP_EOL .
Copy link
Contributor

Choose a reason for hiding this comment

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

Line exceeds 120 characters; contains 126 characters

$this->assertRegExp('/\s*up 20160103083300 2016-01-11 23:53:36 2016-01-11 23:53:37 *\*\* MISSING \*\*'.PHP_EOL.
'\s*up 20160815145812 2016-01-16 18:35:40 2016-01-16 18:35:41 Example *\*\* MISSING \*\*'.PHP_EOL.
'\s*down 20160111235330 Foo\\\\Bar\\\\TestMigration'.PHP_EOL.
$this->assertRegExp('/\s*up 20160103083300 2016-01-11 23:53:36 2016-01-11 23:53:37 *\*\* MISSING \*\*' . PHP_EOL .
Copy link
Contributor

Choose a reason for hiding this comment

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

Line exceeds 120 characters; contains 126 characters

'\s*down 20150111235330 Baz\\\\TestMigration'.PHP_EOL.
'\s*down 20150116183504 Baz\\\\TestMigration2'.PHP_EOL.
'\s*down 20160111235330 Foo\\\\Bar\\\\TestMigration'.PHP_EOL.
$this->assertRegExp('/\s*up 20160103083300 2016-01-11 23:53:36 2016-01-11 23:53:37 *\*\* MISSING \*\*' . PHP_EOL .
Copy link
Contributor

Choose a reason for hiding this comment

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

Line exceeds 120 characters; contains 126 characters

// note that the order is important: missing migrations should appear before down migrations
$this->assertRegExp('/\s*up 20120111235330 2012-01-16 18:35:40 2012-01-16 18:35:41 TestMigration'.PHP_EOL.
'\s*up 20120116183504 2012-01-16 18:35:40 2012-01-16 18:35:41 TestMigration2'.PHP_EOL.
$this->assertRegExp('/\s*up 20120111235330 2012-01-16 18:35:40 2012-01-16 18:35:41 TestMigration' . PHP_EOL .
Copy link
Contributor

Choose a reason for hiding this comment

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

Line exceeds 120 characters; contains 121 characters

// note that the order is important: missing migrations should appear before down migrations
$this->assertRegExp('/\s*up 20160111235330 2016-01-16 18:35:40 2016-01-16 18:35:41 Foo\\\\Bar\\\\TestMigration'.PHP_EOL.
'\s*up 20160116183504 2016-01-16 18:35:40 2016-01-16 18:35:41 Foo\\\\Bar\\\\TestMigration2'.PHP_EOL.
$this->assertRegExp('/\s*up 20160111235330 2016-01-16 18:35:40 2016-01-16 18:35:41 Foo\\\\Bar\\\\TestMigration' . PHP_EOL .
Copy link
Contributor

Choose a reason for hiding this comment

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

Line exceeds 120 characters; contains 135 characters

// place with regard to other up non-missing migrations)
$this->assertRegExp('/\s*up 20120103083300 2012-01-11 23:53:36 2012-01-11 23:53:37 *\*\* MISSING \*\*'.PHP_EOL.
'\s*up 20120111235330 2012-01-16 18:35:40 2012-01-16 18:35:41 TestMigration'.PHP_EOL.
$this->assertRegExp('/\s*up 20120103083300 2012-01-11 23:53:36 2012-01-11 23:53:37 *\*\* MISSING \*\*' . PHP_EOL .
Copy link
Contributor

Choose a reason for hiding this comment

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

Line exceeds 120 characters; contains 126 characters

// place with regard to other up non-missing migrations)
$this->assertRegExp('/\s*up 20160103083300 2016-01-11 23:53:36 2016-01-11 23:53:37 *\*\* MISSING \*\*'.PHP_EOL.
'\s*up 20160111235330 2016-01-16 18:35:40 2016-01-16 18:35:41 Foo\\\\Bar\\\\TestMigration'.PHP_EOL.
$this->assertRegExp('/\s*up 20160103083300 2016-01-11 23:53:36 2016-01-11 23:53:37 *\*\* MISSING \*\*' . PHP_EOL .
Copy link
Contributor

Choose a reason for hiding this comment

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

Line exceeds 120 characters; contains 126 characters

'\s*up 20150111235330 2015-01-16 18:35:40 2015-01-16 18:35:41 Baz\\\\TestMigration'.PHP_EOL.
'\s*down 20150116183504 Baz\\\\TestMigration2'.PHP_EOL.
'\s*down 20160111235330 Foo\\\\Bar\\\\TestMigration'.PHP_EOL.
$this->assertRegExp('/\s*up 20120103083300 2012-01-11 23:53:36 2012-01-11 23:53:37 *\*\* MISSING \*\*' . PHP_EOL .
Copy link
Contributor

Choose a reason for hiding this comment

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

Line exceeds 120 characters; contains 126 characters

@codecov-io
Copy link

codecov-io commented Oct 16, 2017

Codecov Report

Merging #1211 into master will not change coverage.
The diff coverage is 87.5%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1211   +/-   ##
=======================================
  Coverage   72.18%   72.18%           
=======================================
  Files          35       35           
  Lines        5584     5584           
=======================================
  Hits         4031     4031           
  Misses       1553     1553
Impacted Files Coverage Δ
src/Phinx/Db/Adapter/SQLiteAdapter.php 86.65% <0%> (ø) ⬆️
src/Phinx/Db/Adapter/PdoAdapter.php 87.36% <100%> (ø) ⬆️
src/Phinx/Db/Adapter/MysqlAdapter.php 97.84% <100%> (ø) ⬆️
src/Phinx/Db/Adapter/PostgresAdapter.php 92.78% <100%> (ø) ⬆️
src/Phinx/Migration/Manager.php 89.88% <90.9%> (ø) ⬆️

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 81d7f20...1e5db2c. Read the comment docs.

@ADmad ADmad merged commit 8b77502 into cakephp:master Oct 16, 2017
@Iandenh Iandenh deleted the patch-4 branch October 16, 2017 16:04
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.

4 participants