I am working on one of the task mentioned in #1518116: [meta] Make Core pass Coder Review using Code Review module. But recently 'system' table has been removed in D8 [#1813642] and I am not able to use existing patch #1536122: Drupal 8 port as mentioned there.

As I have to complete my task using Code Review module so I have created a new patch to take care of all these issues myself. This patch takes care of {system} table issue and other minor issues.

Apart from this, I am getting this warning "Warning: filemtime(): stat failed for modules/contrib/coder/coder_upgrade/tests/new/samples/example.info in _system_rebuild_module_data() (line 2786 of core/modules/system/system.module)." in Coder Upgrade module. To resolve this, I have can created a blank/empty file, example.info, in coder/coder_upgrade/tests/new/samples/ folder.

This might help to those who want to use Code Review module in D8. You just need to clone 7.x-1.2 branch and apply attached patch to use Code Review in D8.

CommentFileSizeAuthor
coder-7x2x-8x1x.patch5.24 KBamitgoyal
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

douggreen’s picture

Status: Needs review » Fixed

Thanks! I had to make a few tweaks, but this was mostly it. Committed in http://drupalcode.org/project/coder.git/commit/035b9cd.

douggreen’s picture

I'd like the release to show up on the project page, but it's not. If anyone who knows how to fix this, please say so. I'm not sure if the problem is just a decision by the Infra team to not show 8.x releases.

But to help people find the release, here it is http://drupal.org/node/1903494.

solotandem’s picture

Just check the "Show snapshot release" box on administer releases page.

Status: Fixed » Closed (fixed)

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