Support from Acquia helps fund testing for Drupal Acquia logo

Comments

plopesc’s picture

Status: Active » Needs review
FileSize
21.25 KB

Hello

Attaching patch that moves block_test_views to tests/modules folder.
I found that block_test module was placed in tests folder. Then I moved it to tests/modules/block_test folder.

If you consider it shold be moved to tests folder again, just write it ;)

Regards

Status: Needs review » Needs work

The last submitted patch, move_tests-2031881-1.patch, failed testing.

plopesc’s picture

Status: Needs work » Needs review
FileSize
21.4 KB

Rerolling, given that http://drupalcode.org/project/drupal.git/commit/a4c65be modified files affected by this issue.

Regards.

ParisLiakos’s picture

hi, thanks for the patch!
could you maybe use
git diff -M when rolling it? it makes reviewing a lot easier and size a lot smaller:)
thanks again for your time!

plopesc’s picture

FileSize
4.41 KB

Hello

Thanks for your point, I've learned a new git diff option ;)

Here is the patch re-rolled.

Regards.

ParisLiakos’s picture

Status: Needs review » Reviewed & tested by the community

looks great, thanks:)

ParisLiakos’s picture

doublepost.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 9e4991d and pushed to 8.x. Thanks!

Automatically closed -- issue fixed for 2 weeks with no activity.